Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9261 )
Change subject: cmeta: Use more efficient form of GetConsensusRole() ...................................................................... cmeta: Use more efficient form of GetConsensusRole() Now that there is a form of GetConsensusRole() that can be called without instantiating an instance of ConsensusStatePB, which requires a bunch of string copies, let's use that for ConsensusMetadata::UpdateActiveRole() instead. Change-Id: Ia192d75ffef614e2c1832df85c1332cb13d49f39 Reviewed-on: http://gerrit.cloudera.org:8080/9261 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/consensus/consensus_meta.cc 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9261 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia192d75ffef614e2c1832df85c1332cb13d49f39 Gerrit-Change-Number: 9261 Gerrit-PatchSet: 4 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>