Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19923 )
Change subject: [master] Exclude tservers in MAINTENANCE_MODE when leader rebalancing ...................................................................... [master] Exclude tservers in MAINTENANCE_MODE when leader rebalancing Currently, auto leader rebalancing may transfer new leaders to tservers which have entered MAINTENANCE_MODE. Because a tserver in MAINTENANCE_MODE is being maintained by administrators, maybe it is decommissioning. Since it might be decommissioned/decommissioning, it doesn't make sense for the leader rebalancer to transfer a leader to a tserver in MAINTENANCE_MODE as leaders serve many users' requests. For this reason, we should exclude such tservers. This patch fixes the problem and adds an unit test for this. Change-Id: I2f85a675e69fd02a62e2625881dad2ca5e27acd9 Reviewed-on: http://gerrit.cloudera.org:8080/19608 Tested-by: Kudu Jenkins Reviewed-by: Yingchun Lai <[email protected]> Reviewed-by: Yuqi Du <[email protected]> Reviewed-on: http://gerrit.cloudera.org:8080/19923 Tested-by: Yingchun Lai <[email protected]> --- M src/kudu/master/auto_leader_rebalancer-test.cc M src/kudu/master/auto_leader_rebalancer.cc M src/kudu/master/auto_leader_rebalancer.h 3 files changed, 97 insertions(+), 2 deletions(-) Approvals: Yingchun Lai: Looks good to me, approved; Verified Yuqi Du: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/19923 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I2f85a675e69fd02a62e2625881dad2ca5e27acd9 Gerrit-Change-Number: 19923 Gerrit-PatchSet: 3 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
