Hello Tidy Bot, Mike Percy,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8679
to look at the new patch set (#7).
Change subject: KUDU-1097: more robust criteria for replica eviction
......................................................................
KUDU-1097: more robust criteria for replica eviction
Updated the replica eviction criteria to address scenarios where newly
added replicas fail (before or after promotion) one after another
prior to eviction of the original failed voter replica. With this
patch, the replacement process should not end up with placing failed
replicas at every available tablet server, deadlocking the replacement
process. Added corresponding unit tests to cover the updated behavior
of the consensus::CanEvictReplica() utility function.
Updated the replica eviction criteria for more robust handling of the
'replace' attribute. In this context, the case when the leader replica
itself is marked with the 'replace' attribute is now handled properly.
Added unit tests to cover appropriate cases.
Also, enabled the AdminCliTest.TestMoveTablet_KUDU_1097 test scenario
since now it should not be flaky anymore.
Change-Id: I2062c2963fface43062b3154bf5ffa61b4fa684f
---
M src/kudu/consensus/quorum_util-test.cc
M src/kudu/consensus/quorum_util.cc
M src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
M src/kudu/tools/kudu-admin-test.cc
4 files changed, 758 insertions(+), 91 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/8679/7
--
To view, visit http://gerrit.cloudera.org:8080/8679
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2062c2963fface43062b3154bf5ffa61b4fa684f
Gerrit-Change-Number: 8679
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Tidy Bot
