Hello Mahesh Reddy, 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 (#8).
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, 41 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/21073/8
--
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: 8
Gerrit-Owner: Song Jiacheng <songjiach...@thinkingdata.cn>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Tidy Bot (241)
