Pavel Ivanov <[email protected]> writes: > On Mon, Mar 25, 2013 at 4:30 AM, Kristian Nielsen > <[email protected]> wrote: >> Pavel Ivanov <[email protected]> writes: >> >>> fixes. Meanwhile I've found one more bug in your code. In >>> Gtid_log_event::do_apply_event() you assume that mysql_parse() always >>> succeeds, you don't check thd->is_error() and always return 0. I don't
>> Can you test the appended patch if it fixes the problem? I tried to copy the > Yes, this patch works. > And I've found two more minor changes that should be done in GTID > support: gcc -O2 complains that err in > slave_connection_state::remove() in sql/rpl_gtid.cc is assigned but > never used (it's used only in DBUG_ASSERT). Also it complains that > error_gtid in mysql_binlog_send in sql/sql_repl.cc can be used > uninitialized (I see that it can't but it breaks build with -Werror). > And more minor fixes: tests rpl.rpl_gtid_crash and > rpl.rpl_gtid_errorhandling need "--source include/have_debug.inc" and > multi_source.gtid needs "--source include/not_embedded.inc". Thanks. These should all be fixed with my latest push to 10.0-mdev26. - Kristian. _______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~maria-developers More help : https://help.launchpad.net/ListHelp
