Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22387 )
Change subject: [consensus] micro-optimisation on RequestVote() ...................................................................... [consensus] micro-optimisation on RequestVote() This patch simplifies the logic around 'update_guard' in RaftConsensus::RequestVote() since unique_lock<>::try_lock() already reports if the underlying locking primitive has been acquired. It also fixes compilation warning issued by GCC13/14 about ignoring the return value of the unique_lock<>::try_lock() method. This patch doesn't contain any functional modifications. Change-Id: I115d88b90ce6ec94737bb85f1003cf943e1ab9e0 Reviewed-on: http://gerrit.cloudera.org:8080/22387 Tested-by: Kudu Jenkins Reviewed-by: Abhishek Chennaka <[email protected]> --- M src/kudu/consensus/raft_consensus.cc 1 file changed, 8 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22387 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I115d88b90ce6ec94737bb85f1003cf943e1ab9e0 Gerrit-Change-Number: 22387 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]>
