Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15850 )

Change subject: KUDU-3113: fix auto-rebalancer move execution
......................................................................


Patch Set 5:

(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());
> Is there any significance if adding follower_unavailable_considered_failed_
Yeah, it's to wait until the tserver is deemed unavailable by the master -- 
that's what prevents moves from being scheduled in the rebalancer.



--
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: Wed, 06 May 2020 18:11:38 +0000
Gerrit-HasComments: Yes

Reply via email to