Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17565 )
Change subject: consensus: make some methods void ...................................................................... consensus: make some methods void Some methods never returned anything but Status::OK(), at which point they might as well be void methods. This patch contains no major functional changes. Change-Id: Idebe5bae816788e6011c24d6c49f71d9c78ddbf7 Reviewed-on: http://gerrit.cloudera.org:8080/17565 Tested-by: Kudu Jenkins Reviewed-by: Bankim Bhavsar <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/consensus/consensus_peers-test.cc M src/kudu/consensus/consensus_peers.cc M src/kudu/consensus/consensus_peers.h M src/kudu/consensus/peer_manager.cc 4 files changed, 51 insertions(+), 54 deletions(-) Approvals: Kudu Jenkins: Verified Bankim Bhavsar: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17565 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Idebe5bae816788e6011c24d6c49f71d9c78ddbf7 Gerrit-Change-Number: 17565 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
