Skip to content

Commit

Permalink
Use master_retry_count to fix rpl.rpl_semi_sync_master_crash_if_activ…
Browse files Browse the repository at this point in the history
…e_trxs

Summary:
The proper variable to adjust is master_retry_count, not
master_connect_retry.

Squash with D5841339

Reviewed By: abhinav04sharma

Differential Revision: D7317422

fbshipit-source-id: 6938715
  • Loading branch information
Herman Lee authored and facebook-github-bot committed Mar 18, 2018
1 parent 57bfabe commit f5085f4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ call mtr.add_suppression("Force shutdown: Semi-sync master is being switched off
create table t1(a int) engine=innodb;
include/stop_slave.inc
SET GLOBAL DEBUG='+d,dont_send_semi_sync_reply';
CHANGE MASTER TO MASTER_CONNECT_RETRY=0;
CHANGE MASTER TO MASTER_RETRY_COUNT=0;
include/start_slave.inc
insert into t1 values(1);
ERROR HY000: Lost connection to MySQL server during query
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ sync_slave_with_master;
connection slave;
source include/stop_slave.inc;
SET GLOBAL DEBUG='+d,dont_send_semi_sync_reply';
CHANGE MASTER TO MASTER_CONNECT_RETRY=0;
CHANGE MASTER TO MASTER_RETRY_COUNT=0;
source include/start_slave.inc;

##
Expand Down

0 comments on commit f5085f4

Please sign in to comment.