Gabriella Lotz has uploaded a new patch set (#16) to the change originally created by Song Jiacheng. ( http://gerrit.cloudera.org:8080/21073 )
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-test.cc M src/kudu/master/auto_rebalancer.cc M src/kudu/master/auto_rebalancer.h 3 files changed, 109 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/21073/16 -- 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: 16 Gerrit-Owner: Song Jiacheng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Song Jiacheng <[email protected]> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <[email protected]>
