Hello Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/18132
to look at the new patch set (#3).
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/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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/32/18132/3
--
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: newpatchset
Gerrit-Change-Id: I4e51349d12fffb54458142169b9e7789c8d3987c
Gerrit-Change-Number: 18132
Gerrit-PatchSet: 3
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
