Yifan Zhang has uploaded this change for review. ( 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 --- 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 4 files changed, 107 insertions(+), 80 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/32/18132/1 -- 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: newchange Gerrit-Change-Id: I4e51349d12fffb54458142169b9e7789c8d3987c Gerrit-Change-Number: 18132 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]>
