Hello Nirbhay, On 01/09/2017 09:48 PM, Nirbhay Choubey wrote:
Hi Bar,
On Mon, Jan 9, 2017 at 1:00 AM, Alexander Barkov <bar@mariadb.org <mailto:bar@mariadb.org>> wrote:
Hello Nirbhay,
IIRC, you was working on this issue. What is the current status?
Yes, I have just committed a patch. Could you please take a look at it?
The patch http://lists.askmonty.org/pipermail/commits/2017-January/010405.html looks Ok. I have a proposal for tests. The problem is that utf32, utf16 and ucs2 are not always compiled-in. So in case when some of the affected charsets is missing, sql_mode.test will fail. Can you please move the tests into ctype_utf32.test, ctype_utf16.test, ctype_ucs2.test ? I also suggest to add "SELECT @@sql_mode", to make sure that the assignment actually does what it's supposed to do. Remembering sql_mode is not necessary. It's OK to restore it to DEFAULT. Remembering character_set_connection is not necessary. It's Ok to do "SET NAMES utf8" at the end. Collecting all together, I'd suggest this chuck for ctype_utf32.test: --echo # --echo # MDEV-11685: sql_mode can't be set with non-ascii connection charset --echo # SET character_set_connection=utf32; SET sql_mode='NO_ENGINE_SUBSTITUTION'; SELECT @@sql_mode; SET sql_mode=DEFAULT; SET NAMES utf8; and similar chunks for ctype_utf16.test and ctype_ucs2.test. Thanks!
Best, Nirbhay
Thanks!
On 01/03/2017 09:31 PM, Elena Stepanova (JIRA) wrote: > > [ https://jira.mariadb.org/browse/MDEV-11685?page=com.atlassian.jira.plugin.sy... <https://jira.mariadb.org/browse/MDEV-11685?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel> ] > > Elena Stepanova reassigned MDEV-11685: > -------------------------------------- > > Labels: upstream (was: ) > Affects Version/s: 5.5 > 10.0 > 10.1 > 10.2 > Assignee: Alexander Barkov > >> sql_mode can't be set with non-ascii connection charset >> ------------------------------------------------------- >> >> Key: MDEV-11685 >> URL: https://jira.mariadb.org/browse/MDEV-11685 <https://jira.mariadb.org/browse/MDEV-11685> >> Project: MariaDB Server >> Issue Type: Bug >> Components: Character Sets >> Affects Versions: 10.2.3, 5.5, 10.0, 10.1, 10.2 >> Reporter: Nirbhay Choubey >> Assignee: Alexander Barkov >> Labels: upstream >> >> {code} >> MariaDB [test]> set sql_mode = 'NO_ENGINE_SUBSTITUTION'; >> Query OK, 0 rows affected (0.00 sec) >> MariaDB [test]> set character_set_connection=utf32; >> Query OK, 0 rows affected (0.00 sec) >> MariaDB [test]> set sql_mode = 'NO_ENGINE_SUBSTITUTION'; >> ERROR 1231 (42000): Variable 'sql_mode' can't be set to the value of 'NO_ENGINE_SUBSTITUTION' >> {code} > > > > -- > This message was sent by Atlassian JIRA > (v7.2.1#72003) >