Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9487 )
Change subject: [consensus] fix on RequestForPeer clean-up ...................................................................... [consensus] fix on RequestForPeer clean-up The consensus queue can switch to the non-leader mode when exiting from the RequestForPeer() method scope, so it's necessary to check for that condition in the scoped clean-up object before updating information on the peer's health. The issue was found while running the raft_consensus_stress-itest for the 3-2-3 re-replication scheme, so I don't think any additional test is needed to cover this change. Change-Id: I71b3b47bbf9b5011bd11534287f6ae8f760addd5 Reviewed-on: http://gerrit.cloudera.org:8080/9487 Tested-by: Kudu Jenkins Reviewed-by: Mike Percy <mpe...@apache.org> --- M src/kudu/consensus/consensus_queue.cc 1 file changed, 11 insertions(+), 2 deletions(-) Approvals: Kudu Jenkins: Verified Mike Percy: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9487 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I71b3b47bbf9b5011bd11534287f6ae8f760addd5 Gerrit-Change-Number: 9487 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>