Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9786 )
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 (cherry picked from commit fcb0be6381a47155e171eb50a333af502bbf506f) Reviewed-on: http://gerrit.cloudera.org:8080/9786 --- 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 -- To view, visit http://gerrit.cloudera.org:8080/9786 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: merged Gerrit-Change-Id: If77474adc8f618f2cda35f992190133138fdb511 Gerrit-Change-Number: 9786 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Mike Percy <[email protected]>
