Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18132 )
Change subject: [rebalancer_tool] refactor on IgnoredTserversRunner ...................................................................... [rebalancer_tool] refactor on IgnoredTserversRunner ClusterInfo::tservers_to_empty is not needed to be populated when 'BuildClusterInfo' called, we use it in two cases: - print replica count about tservers_to_empty. - get replicas running on tservers_to_empty (used in IgnoredTserverRunner). But 'BuildClusterInfo" could be called in many places. This patch removed it from ClusterInfo so we could get the information only when needed. Change-Id: I4e51349d12fffb54458142169b9e7789c8d3987c Reviewed-on: http://gerrit.cloudera.org:8080/18132 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/rebalance/rebalance_algo.h M src/kudu/rebalance/rebalancer.cc M src/kudu/rebalance/rebalancer.h M src/kudu/tools/rebalancer_tool.cc M src/kudu/tools/rebalancer_tool.h 5 files changed, 106 insertions(+), 83 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18132 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4e51349d12fffb54458142169b9e7789c8d3987c Gerrit-Change-Number: 18132 Gerrit-PatchSet: 4 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]>
