Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14742
Change subject: rebalancer: fix on Runner::UpdateMovesInProgressStatus() ...................................................................... rebalancer: fix on Runner::UpdateMovesInProgressStatus() This patch added a new parameter 'has_pending_moves' for Runner::UpdateMovesInProgressStatus(), see the comments for this method. Before this patch, the return value of this method was ambiguous, and there was a bug in PolicyFixer: When no pending operation completed, the rebalancer tool would re-synchronize the state of cluster. Now, the re-synchronization of the cluster state happens only in case of an error or timeout or no pending movement. Change-Id: I8a25574d13f36a8f4f98f4340aa2086711b1c741 --- M src/kudu/rebalance/rebalancer.h M src/kudu/tools/rebalancer_tool.cc M src/kudu/tools/rebalancer_tool.h 3 files changed, 39 insertions(+), 20 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/14742/1 -- To view, visit http://gerrit.cloudera.org:8080/14742 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8a25574d13f36a8f4f98f4340aa2086711b1c741 Gerrit-Change-Number: 14742 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]>
