Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21998 )
Change subject: [consensus] small clean-up on PendingRounds ...................................................................... [consensus] small clean-up on PendingRounds I was looking at the code around PendingRounds while troubleshooting a race condition, and found that there is room for micro-optimisations and syntactic sugar in the PendingRounds implementation. This patch doesn't contain any functional modifications. Change-Id: Iaa5b66d8a2cf179f43141de8d640251e1a2dc80f Reviewed-on: http://gerrit.cloudera.org:8080/21878 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit b6320038376b6fb35bfd1f1ac74323168df0884a) Reviewed-on: http://gerrit.cloudera.org:8080/21998 --- M src/kudu/consensus/pending_rounds.cc M src/kudu/consensus/pending_rounds.h 2 files changed, 20 insertions(+), 18 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/21998 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: Iaa5b66d8a2cf179f43141de8d640251e1a2dc80f Gerrit-Change-Number: 21998 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)
