Hakan Kuecuekyilmaz пишет:
> At file:///Users/hakan/work/monty_program/maria/
> 
> ------------------------------------------------------------
> revno: 2720
> revision-id: [email protected]
> parent: [email protected]
> committer: Hakan Kuecuekyilmaz <[email protected]>
> branch nick: maria
> timestamp: Wed 2009-09-02 23:46:17 +0200
> message:
>   Fix for
>       mysqlslap: setting --engine does not get replicated
>       http://bugs.mysql.com/bug.php?id=46967
>   
>   Problem was that a "set storage_engine = <storage_engine>"
>   is not replicated.
>   
>   Fixed by adding an explicit
>       Engine = <storage_engine>
>   
>   to the build_table_string() function.
>   
>   Open question:
>     Should MySQL's replication actually replicate the
>     set storage_engine command or not? Answer: no it should
>     not (documented).

The correct question is should be default engine be replicated as
'default' or as certain engine. I think it should be 'default' (it seems
logical for me and gives more space for different setup of master/slave).



> === modified file 'client/mysqlslap.c'
> --- a/client/mysqlslap.c      2009-04-25 10:05:32 +0000
> +++ b/client/mysqlslap.c      2009-09-02 21:46:17 +0000
> @@ -880,6 +880,12 @@
>      }
>
>    dynstr_append(&table_string, ")");
> +
> +  if (default_engine) {

Move above { on a new line, This program should follow mysql code style.

> +    dynstr_append(&table_string, " Engine = ");
> +    dynstr_append(&table_string, default_engine);
> +  }
> +
>    ptr= (statement *)my_malloc(sizeof(statement),
>                                MYF(MY_ZEROFILL|MY_FAE|MY_WME));
>    ptr->string = (char *)my_malloc(table_string.length+1,

_______________________________________________
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