Hannah Nguyen has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14177 )

Change subject: KUDU-2780: create thread for auto-rebalancing
......................................................................


Patch Set 20:

(2 comments)

rify

http://gerrit.cloudera.org:8080/#/c/14177/20/src/kudu/master/auto_rebalancer-test.cc
File src/kudu/master/auto_rebalancer-test.cc:

http://gerrit.cloudera.org:8080/#/c/14177/20/src/kudu/master/auto_rebalancer-test.cc@457
PS20, Line 457: // Verify that movement of replicas to meet the replication 
factor
              : // does not count towards rebalancing, i.e. the auto-rebalancer 
will
              : // not consider recovering replicas as candidates for replica 
movement.
> BTW, how do we know that the absence of attempts to rebalance is not due to
Updated the test accordingly, thank you!


http://gerrit.cloudera.org:8080/#/c/14177/20/src/kudu/master/auto_rebalancer-test.cc@462
PS20, Line 462: FLAGS_tserver_unresponsive_timeout_ms
> It's a separate topic (i.e. not a topic for this particular scenario), but
Added another test case to make sure that the auto-rebalancer handles this 
scenario as expected.



--
To view, visit http://gerrit.cloudera.org:8080/14177
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifca25d1063c07047cf2123e6792b3c7395be20e4
Gerrit-Change-Number: 14177
Gerrit-PatchSet: 20
Gerrit-Owner: Hannah Nguyen <[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-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Sat, 14 Mar 2020 22:31:33 +0000
Gerrit-HasComments: Yes

Reply via email to