Commit 79faadd5 authored by He Zhenxing's avatar He Zhenxing

Backport BUG#12190 CHANGE MASTER has differ path requiremts on MASTER_LOG_FILE and RELAY_LOG_FILE

CHANGE MASTER TO command required the value for RELAY_LOG_FILE to
be an absolute path, which was different from the requirement of
MASTER_LOG_FILE.

This patch fixed the problem by changing the value for RELAY_LOG_FILE
to be the basename of the log file as that for MASTER_LOG_FILE.
parent 86c36aa1
...@@ -69,7 +69,7 @@ let $_fake_relay_log_purge= `SELECT @@global.relay_log_purge`; ...@@ -69,7 +69,7 @@ let $_fake_relay_log_purge= `SELECT @@global.relay_log_purge`;
# Create relay log file. # Create relay log file.
copy_file $fake_relay_log $_fake_relay_log; copy_file $fake_relay_log $_fake_relay_log;
# Create relay log index. # Create relay log index.
--exec echo $_fake_filename-fake.000001 > $_fake_relay_index --exec echo ./$_fake_filename-fake.000001 > $_fake_relay_index
# Setup replication from existing relay log. # Setup replication from existing relay log.
eval CHANGE MASTER TO MASTER_HOST='dummy.localdomain', RELAY_LOG_FILE='$_fake_filename-fake.000001', RELAY_LOG_POS=4; eval CHANGE MASTER TO MASTER_HOST='dummy.localdomain', RELAY_LOG_FILE='$_fake_filename-fake.000001', RELAY_LOG_POS=4;
......
...@@ -26,7 +26,7 @@ let $MYSQLD_DATADIR= `select @@datadir`; ...@@ -26,7 +26,7 @@ let $MYSQLD_DATADIR= `select @@datadir`;
copy_file $MYSQL_TEST_DIR/std_data/bug33029-slave-relay-bin.000001 $MYSQLD_DATADIR/slave-relay-bin.000001; copy_file $MYSQL_TEST_DIR/std_data/bug33029-slave-relay-bin.000001 $MYSQLD_DATADIR/slave-relay-bin.000001;
write_file $MYSQLD_DATADIR/slave-relay-bin.index; write_file $MYSQLD_DATADIR/slave-relay-bin.index;
slave-relay-bin.000001 ./slave-relay-bin.000001
EOF EOF
change master to change master to
......
...@@ -100,3 +100,20 @@ n ...@@ -100,3 +100,20 @@ n
1 1
2 2
drop table t1; drop table t1;
stop slave;
drop table if exists t1,t2,t3,t4,t5,t6,t7,t8,t9;
reset master;
reset slave;
drop table if exists t1,t2,t3,t4,t5,t6,t7,t8,t9;
start slave;
create table t1 (a int);
insert into t1 values (1);
flush logs;
insert into t1 values (2);
include/stop_slave.inc
delete from t1 where a=2;
CHANGE MASTER TO relay_log_file='slave-relay-bin.000005', relay_log_pos=4;
start slave sql_thread;
start slave io_thread;
set global relay_log_purge=1;
drop table t1;
...@@ -33,3 +33,56 @@ connection slave; ...@@ -33,3 +33,56 @@ connection slave;
sync_with_master; sync_with_master;
# End of 4.1 tests # End of 4.1 tests
#
# BUG#12190 CHANGE MASTER has differ path requiremts on MASTER_LOG_FILE and RELAY_LOG_FILE
#
source include/master-slave-reset.inc;
connection master;
create table t1 (a int);
insert into t1 values (1);
flush logs;
insert into t1 values (2);
# Note: the master positon saved by this will also be used by the
# 'sync_with_master' below.
sync_slave_with_master;
# Check if the table t1 and t2 are identical on master and slave;
let $diff_table_1= master:test.t1
let $diff_table_2= slave:test.t1
source include/diff_tables.inc;
connection slave;
source include/stop_slave.inc;
delete from t1 where a=2;
# start replication from the second insert, after fix of BUG#12190,
# relay_log_file does not use absolute path, only the filename is
# required
#
# Note: the follow change master will automatically reset
# relay_log_purge to false, save the old value to restore
let $relay_log_purge= `select @@global.relay_log_purge`;
CHANGE MASTER TO relay_log_file='slave-relay-bin.000005', relay_log_pos=4;
start slave sql_thread;
source include/wait_for_slave_sql_to_start.inc;
# Sync to the same position saved by the 'sync_slave_with_master' above.
sync_with_master;
# Check if the table t1 and t2 are identical on master and slave;
let $diff_table_1= master:test.t1
let $diff_table_2= slave:test.t1
source include/diff_tables.inc;
# clean up
connection slave;
start slave io_thread;
source include/wait_for_slave_io_to_start.inc;
eval set global relay_log_purge=$relay_log_purge;
connection master;
drop table t1;
sync_slave_with_master;
...@@ -1400,9 +1400,11 @@ bool change_master(THD* thd, Master_info* mi) ...@@ -1400,9 +1400,11 @@ bool change_master(THD* thd, Master_info* mi)
if (lex_mi->relay_log_name) if (lex_mi->relay_log_name)
{ {
need_relay_log_purge= 0; need_relay_log_purge= 0;
strmake(mi->rli.group_relay_log_name,lex_mi->relay_log_name, char relay_log_name[FN_REFLEN];
mi->rli.relay_log.make_log_name(relay_log_name, lex_mi->relay_log_name);
strmake(mi->rli.group_relay_log_name, relay_log_name,
sizeof(mi->rli.group_relay_log_name)-1); sizeof(mi->rli.group_relay_log_name)-1);
strmake(mi->rli.event_relay_log_name,lex_mi->relay_log_name, strmake(mi->rli.event_relay_log_name, relay_log_name,
sizeof(mi->rli.event_relay_log_name)-1); sizeof(mi->rli.event_relay_log_name)-1);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment