Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15850 )
Change subject: KUDU-3113: fix auto-rebalancer move execution ...................................................................... Patch Set 5: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/15850/4/src/kudu/master/auto_rebalancer-test.cc File src/kudu/master/auto_rebalancer-test.cc: http://gerrit.cloudera.org:8080/#/c/15850/4/src/kudu/master/auto_rebalancer-test.cc@575 PS4, Line 575: NO_FATALS(cluster_->mini_tablet_server(0)->Shutdown()); : SleepFor(MonoDelta::FromMilliseconds(FLAGS_tserver_unresponsive_timeout_ms)); : ASSERT_OK(cluster_->AddTabletServer()); > Yeah, it's to wait until the tserver is deemed unavailable by the master -- I see. I guess another important point here is that automatic re-replication kicks in only after FLAGS_follower_unavailable_considered_failed_sec interval, which is set to be equal to FLAGS_tserver_unresponsive_timeout_ms. -- To view, visit http://gerrit.cloudera.org:8080/15850 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If658997dc9bcb709c27d981db56cf2db13ba235f Gerrit-Change-Number: 15850 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hannah Nguyen <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 07 May 2020 04:18:01 +0000 Gerrit-HasComments: Yes
