Hi Bar,

On Tue, Jan 10, 2017 at 2:41 AM, Alexander Barkov <bar@mariadb.org> wrote:
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.

Done.

http://lists.askmonty.org/pipermail/commits/2017-January/010414.html

Thanks for the review.

Best,
Nirbhay
 


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.system.issuetabpanels:all-tabpanel
>     <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)
>     >
>
>