Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 )
Change subject: [rebalance] Add '--known_unhealthy_tservers' flag to rebalancer ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13539/1//COMMIT_MSG Commit Message: PS1: > Curious why you went with this approach vs. a boolean flag that just allowe My fear is that the rebalancer would run for a long time if there are many tablets on tservers, in this time if some tservers crashed unexpectedly the rebalance process should exit. So the rebalancer only need to ignore some specified unhealthy servers. -- To view, visit http://gerrit.cloudera.org:8080/13539 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie83a7d2497b778833f3f96fa8aa20e7486c5ebbb Gerrit-Change-Number: 13539 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Comment-Date: Fri, 07 Jun 2019 10:06:10 +0000 Gerrit-HasComments: Yes
