Fix for BUG#13023: "SQL Thread is up but doesn't move forward". Details in slave.cc;
in short we now record whenever the slave I/O thread ignores a master's event because of its server id,
and use this info in the slave SQL thread to advance Exec_master_log_pos. Because if we
do not, this variable stays at the position of the last executed event, i.e. the last *non-ignored*
executed one, which may not be the last of the master's binlog (and so the slave *looks* behind
the master though it's data-wise it's not).
mysql-test/t/rpl_dual_pos_advance-master.opt:
empty; its goal is just to trigger a server restart after running the test,
so that the master forgets that it was a slave (otherwise it affects the following tests).
sql/log.cc:
No more default arguments for Rotate_log_event constructor.
MYSQL_LOG::appendv() is now called without mutex.
sql/log_event.cc:
Moving one Rotate_log_event constructor from log_event.h. Support for on-demand choice of
duplicating the string argument of the constructor or not (because there now are needs for both
alternatives, see slave.cc).
sql/log_event.h:
We now have a case where a Rotate_log_event is executed by the slave SQL thread while
not being in the relay log, so it needs to pretend its length is 0: a ZERO_LEN flag for that;
a flag DUP_NAME (replaces "bool alloced") to be able to choose if we want the constructor
to duplicate the string argument or not.
sql/slave.cc:
A comment for BUG#13861 (to be fixed). llstr() instead of %ld as the number is ulonglong.
mi->rli becomes rli in some places.
Fix for BUG#13023:
- in the slave I/O thread, whenever we ignore an event because of its server id we update
a couple of coordinates in memory
- in the slave SQL thread, whenever we bump into the end of the latest relay log, we check
this couple of coordinates to see if we should advance our Exec_master_log_pos.
- when the slave I/O thread terminates it saves these in-memory coordinates into a Rotate event
in the relay log, so that they are durable.
sql/slave.h:
A couple of coordinates in RELAY_LOG_INFO to keep track of the last ignored events received
by the slave I/O thread (ignored because of the server id).
mysql-test/r/rpl_dual_pos_advance.result:
New BitKeeper file ``mysql-test/r/rpl_dual_pos_advance.result''
mysql-test/t/rpl_dual_pos_advance.test:
Test for BUG#13023 (with a part, disabled, to test BUG#13861 when I fix it).
Before the fix, this test used to hang.
2005-10-12 13:29:55 +02:00
|
|
|
# This test checks that in a dual-head setup
|
|
|
|
# A->B->A, where A has --log-slave-updates (why would it?
|
|
|
|
# assume that there is a C as slave of A),
|
|
|
|
# then the Exec_master_log_pos of SHOW SLAVE STATUS does
|
|
|
|
# not stay too low on B(BUG#13023 due to events ignored because
|
|
|
|
# of their server id).
|
|
|
|
# It also will test BUG#13861.
|
|
|
|
|
|
|
|
source include/master-slave.inc;
|
|
|
|
|
|
|
|
|
|
|
|
# set up "dual head"
|
|
|
|
|
|
|
|
connection slave;
|
|
|
|
reset master;
|
|
|
|
|
|
|
|
connection master;
|
|
|
|
--replace_result $SLAVE_MYPORT SLAVE_PORT
|
|
|
|
eval change master to master_host="127.0.0.1",master_port=$SLAVE_MYPORT,master_user="root";
|
|
|
|
|
|
|
|
start slave;
|
|
|
|
|
|
|
|
# now we test it
|
|
|
|
|
|
|
|
connection slave;
|
|
|
|
|
|
|
|
create table t1 (n int);
|
|
|
|
|
|
|
|
save_master_pos;
|
|
|
|
connection master;
|
|
|
|
sync_with_master;
|
|
|
|
|
|
|
|
# Now test BUG#13861. This will be enabled when Guilhem fixes this
|
|
|
|
# bug.
|
|
|
|
|
|
|
|
# stop slave
|
|
|
|
|
|
|
|
# create table t2 (n int); # create one ignored event
|
|
|
|
|
|
|
|
# save_master_pos;
|
|
|
|
# connection slave;
|
|
|
|
# sync_with_master;
|
|
|
|
|
|
|
|
# connection slave;
|
|
|
|
|
|
|
|
# show tables;
|
|
|
|
|
|
|
|
# save_master_pos;
|
|
|
|
|
|
|
|
# create table t3 (n int);
|
|
|
|
|
|
|
|
# connection master;
|
|
|
|
|
|
|
|
# bug is that START SLAVE UNTIL may stop too late, we test that by
|
|
|
|
# asking it to stop before creation of t3.
|
|
|
|
|
|
|
|
# start slave until master_log_file="slave-bin.000001",master_log_pos=195;
|
|
|
|
|
|
|
|
# wait until it's started (the position below is the start of "CREATE
|
|
|
|
# TABLE t2") (otherwise wait_for_slave_to_stop may return at once)
|
|
|
|
|
|
|
|
# select master_pos_wait("slave-bin.000001",137);
|
|
|
|
|
|
|
|
# wait_for_slave_to_stop;
|
|
|
|
|
|
|
|
# then BUG#13861 causes t3 to show up below (because stopped too
|
|
|
|
# late).
|
|
|
|
|
|
|
|
# show tables;
|
|
|
|
|
|
|
|
# start slave;
|
|
|
|
|
|
|
|
# BUG#13023 is that Exec_master_log_pos may stay too low "forever":
|
|
|
|
|
|
|
|
connection master;
|
|
|
|
|
|
|
|
create table t4 (n int); # create 3 ignored events
|
|
|
|
create table t5 (n int);
|
|
|
|
create table t6 (n int);
|
|
|
|
|
|
|
|
save_master_pos;
|
|
|
|
connection slave;
|
|
|
|
sync_with_master;
|
|
|
|
|
|
|
|
connection slave;
|
|
|
|
|
|
|
|
save_master_pos;
|
|
|
|
|
|
|
|
connection master;
|
|
|
|
|
|
|
|
# then BUG#13023 caused hang below ("master" looks behind, while it's
|
|
|
|
# not in terms of updates done).
|
|
|
|
|
|
|
|
sync_with_master;
|
|
|
|
|
|
|
|
show tables;
|
|
|
|
|
|
|
|
# cleanup
|
|
|
|
|
|
|
|
stop slave;
|
|
|
|
reset slave;
|
|
|
|
drop table t1,t4,t5,t6; # add t2 and t3 later
|
|
|
|
|
|
|
|
save_master_pos;
|
|
|
|
connection slave;
|
|
|
|
sync_with_master;
|
|
|
|
|
|
|
|
# End of 4.1 tests
|
2007-03-20 11:15:15 +01:00
|
|
|
|
|
|
|
# Cleanup
|
|
|
|
# The A->B->A replication causes the master to start writing relay logs
|
|
|
|
# in var/run, remove them
|
|
|
|
remove_file $MYSQLTEST_VARDIR/run/master-relay-bin.000001;
|
|
|
|
remove_file $MYSQLTEST_VARDIR/run/master-relay-bin.index;
|