Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21073 to look at the new patch set (#6). Change subject: Remove the replace flag if the move fails in auto rebalancing. ...................................................................... Remove the replace flag if the move fails in auto rebalancing. If a move fails because of some reasons, like high workload, the auto rebalancer will remove the ReplicaMove from the vector and just skip it to next, leaving the replace flag still true. And then the master will add a new peer to the tablet, trying to replace the original replica. But if the replica is a leader, it will never be replaced since no one will send LeaderStepDown request to it. So remove the replace flag of the original replica before removing the move, rollback the modification if it fails. Change-Id: I99dafa654878b9d6d8162d84500913ae0655692b --- M src/kudu/master/auto_rebalancer.cc M src/kudu/master/auto_rebalancer.h M src/kudu/rebalance/rebalancer.h 3 files changed, 38 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/21073/6 -- To view, visit http://gerrit.cloudera.org:8080/21073 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I99dafa654878b9d6d8162d84500913ae0655692b Gerrit-Change-Number: 21073 Gerrit-PatchSet: 6 Gerrit-Owner: Song Jiacheng <songjiach...@thinkingdata.cn> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)