Hello Tidy Bot, 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 (#3). 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, 288 insertions(+), 110 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/8245/3 -- 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: 3 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>