Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10821 )
Change subject: consensus queue: Clarify return value of ResponseFromPeer() ...................................................................... consensus queue: Clarify return value of ResponseFromPeer() This patch simply makes some minor semantic improvements to the ConsensusQueue::ResponseFromPeer() API by returning the value of whether to immediately send another request and improving the header file comment documentation about those semantics. Change-Id: Ibe73b5c69630898327ac7787e3dfcb0b761f323e Reviewed-on: http://gerrit.cloudera.org:8080/10821 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- 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 4 files changed, 89 insertions(+), 79 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10821 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ibe73b5c69630898327ac7787e3dfcb0b761f323e Gerrit-Change-Number: 10821 Gerrit-PatchSet: 4 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot
