diff options
author | unknown <Dao-Gang.Qu@sun.com> | 2009-12-31 12:04:19 +0800 |
---|---|---|
committer | unknown <Dao-Gang.Qu@sun.com> | 2009-12-31 12:04:19 +0800 |
commit | be397eb40061afdc924cabb96efca0ec736b5181 (patch) | |
tree | e3c946bce1ed210af4829507affba2df88dbf061 /sql/sql_parse.cc | |
parent | 041baac3262a8d42452c83216d790ef1557668c3 (diff) | |
parent | 2b2ce3d6cb01a36cd35191e8670dcb023420c84e (diff) | |
download | mariadb-git-be397eb40061afdc924cabb96efca0ec736b5181.tar.gz |
Bug #49137 Replication failure on SBR/MBR + multi-table DROP TEMPORARY TABLE
In statement-based or mixed-mode replication, use DROP TEMPORARY TABLE
to drop multiple tables causes different errors on master and slave,
when one or more of these tables do not exist. Because when executed
on slave, it would automatically add IF EXISTS to the query to ignore
all ER_BAD_TABLE_ERROR errors.
To fix the problem, do not add IF EXISTS when executing DROP TEMPORARY
TABLE on the slave, and clear the ER_BAD_TABLE_ERROR error after
execution if the query does not expect any errors.
mysql-test/suite/rpl/r/rpl_drop_temp.result:
Updated for the patch of bug#49137.
mysql-test/suite/rpl/t/rpl_drop_temp.test:
Added the test file to verify if DROP MULTI TEMPORARY TABLE
will cause different errors on master and slave, when one or
more of these tables do not exist.
sql/log_event.cc:
Added code to handle above cases which are
removed from sql_parse.cc
sql/sql_parse.cc:
Remove the code to issue the 'Unknown table' error,
if the temporary table does not exist when dropping
it on slave. The above cases decribed in comments
will be handled later in log_event.cc.
Diffstat (limited to 'sql/sql_parse.cc')
-rw-r--r-- | sql/sql_parse.cc | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/sql/sql_parse.cc b/sql/sql_parse.cc index 4f1524c1dc8..69c9ddc7806 100644 --- a/sql/sql_parse.cc +++ b/sql/sql_parse.cc @@ -3373,17 +3373,6 @@ end_with_restore_list: } else { - /* - If this is a slave thread, we may sometimes execute some - DROP / * 40005 TEMPORARY * / TABLE - that come from parts of binlogs (likely if we use RESET SLAVE or CHANGE - MASTER TO), while the temporary table has already been dropped. - To not generate such irrelevant "table does not exist errors", - we silently add IF EXISTS if TEMPORARY was used. - */ - if (thd->slave_thread) - lex->drop_if_exists= 1; - /* So that DROP TEMPORARY TABLE gets to binlog at commit/rollback */ thd->options|= OPTION_KEEP_LOG; } |