Hello Mike Percy, David Ribeiro Alves, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8245
to look at the new patch set (#2).
Change subject: WIP: KUDU-2048. consensus: only evict unresponsive nodes if
remaining voters are viable
......................................................................
WIP: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters
are viable
This adds a heuristic to the leader when it is deciding whether to evict
an unresponsive node. It will now only do so if the remaining nodes
appear viable (i.e they have recently had a successful communication
with the leader).
WIP: trying to do some cleanup of the error tracking -- it seems that
consensus_queue.cc doesn't get any notification of when a follower is
getting 'connection refused' errors etc, which isn't great for this
heuristic. Posting to gerrit for a full test run.
Change-Id: I673f5b8a58b3954ea28066ecb334b3fdd60e7adb
---
M src/kudu/consensus/consensus_peers.cc
M src/kudu/consensus/consensus_queue-test.cc
M src/kudu/consensus/consensus_queue.cc
M src/kudu/consensus/consensus_queue.h
M src/kudu/integration-tests/tablet_replacement-itest.cc
5 files changed, 193 insertions(+), 72 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/8245/2
--
To view, visit http://gerrit.cloudera.org:8080/8245
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I673f5b8a58b3954ea28066ecb334b3fdd60e7adb
Gerrit-Change-Number: 8245
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
