Hi, Sergey!

On Jul 17, Sergey Petrunya wrote:
> ------------------------------------------------------------
> revno: 3453
> revision-id: [email protected]
> parent: [email protected]
> committer: Sergey Petrunya <[email protected]>
> branch nick: 5.5-show-explain-r21
> timestamp: Tue 2012-07-17 23:27:03 +0400
> message:
>   MWL#182: Explain running statements
>   Address feedback from the second code review:
>   - Make SHOW EXPLAIN code convert the query string before pushing it as a 
> warning text.   
> @@ -2135,8 +2135,35 @@ int fill_show_explain(THD *thd, TABLE_LI
>      }
>      else
>      {
> +      /*
> +        Push the query string as a warning. The query may be in a different
> +        charset than the charset that's used for error messages, so, convert 
> it
> +        if needed.
> +      */

there's no test case for this. was it too difficult to create?

Regards,
Sergei

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to