Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18114
Change subject: KUDU-3346: fix rebalancer tool fails to run with '--ignored_tservers' ...................................................................... KUDU-3346: fix rebalancer tool fails to run with '--ignored_tservers' Before this patch, the validity of 'ignored_tservers' was checked in 'BuildClusterinfo', which leads to a failure when the 'raw_info' only contains information of tservers on a specific location. This patch fix it by moving the parameter validity check into 'KsckResultsToClusterRawInfo' because ksck results always contains all tservers in the cluster. And also, I add 'needs_tservers_to_empty_info' to 'BuildClusterInfo', we only need this information when: - print the state of whole cluster before and after the rebalancer runs. - get the state of whole cluster during running IgnoredTServersRunner. In these two cases 'raw_info' contains all tablet servers info in the cluster so every ignored tservers could be found in 'raw_info'. Change-Id: I1361f562f3e886077a79c3de8ea5fb2ebb8df6e9 --- M src/kudu/rebalance/rebalance-test.cc M src/kudu/rebalance/rebalancer.cc M src/kudu/rebalance/rebalancer.h M src/kudu/tools/rebalancer_tool-test.cc M src/kudu/tools/rebalancer_tool.cc M src/kudu/tools/rebalancer_tool.h 6 files changed, 189 insertions(+), 51 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/18114/1 -- To view, visit http://gerrit.cloudera.org:8080/18114 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I1361f562f3e886077a79c3de8ea5fb2ebb8df6e9 Gerrit-Change-Number: 18114 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]>
