Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9267 )
Change subject: Random cleanups in RaftConsensus ...................................................................... Random cleanups in RaftConsensus This patch implements various random cleanups in RaftConsensus and ConsensusMetadata, including adding a DCHECK and an AssertWaitAllowed(), in addition to only acquiring the RaftConsensus lock once in the election callback. Change-Id: Ie470a526a4a0b0595b3d4310a961876e52a99b0a Reviewed-on: http://gerrit.cloudera.org:8080/9267 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/consensus/consensus_meta.h M src/kudu/consensus/raft_consensus.cc 2 files changed, 35 insertions(+), 37 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9267 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie470a526a4a0b0595b3d4310a961876e52a99b0a Gerrit-Change-Number: 9267 Gerrit-PatchSet: 3 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
