Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9755 )
Change subject: [consensus] KUDU-2367 fix replica health report ...................................................................... [consensus] KUDU-2367 fix replica health report This patch addresses KUDU-2367,where a leader replica under particular conditions reports follower's health status as FAILED instead of FAILED_UNRECOVERABLE. I.e., the bug in the code of the PeerMessageQueue::PeerHealthStatus() method resulted in having the leader reporting the health status of the replica still as FAILED even after the replica fell behind the WAL segment GC threshold. Also, a couple of already existing integration tests are updated to provide appropriate coverage for this patch. Change-Id: If77474adc8f618f2cda35f992190133138fdb511 Reviewed-on: http://gerrit.cloudera.org:8080/9755 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- 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/raft_consensus-itest-base.cc M src/kudu/integration-tests/raft_consensus-itest-base.h M src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc 6 files changed, 137 insertions(+), 25 deletions(-) Approvals: Alexey Serbin: Verified Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9755 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If77474adc8f618f2cda35f992190133138fdb511 Gerrit-Change-Number: 9755 Gerrit-PatchSet: 8 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
