Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17489
Change subject: [consensus] a small optimisation on log prefix ...................................................................... [consensus] a small optimisation on log prefix This patch introduces a small optimisation into the implementation of the RaftConsensus::LogPrefixThreadSafe() method: since the prefix it produces doesn't change, it's possible to construct the string once in the constructor and then return a constant reference. I also updated PendingRound class to store a reference to the prefix as well since instances of the PendingRound class have shorter live span than RaftConsensus objects which create PendingRound objects. Change-Id: I6a5082fe07449ef24827696638ddd8e925a08264 --- M src/kudu/consensus/pending_rounds.cc M src/kudu/consensus/pending_rounds.h M src/kudu/consensus/raft_consensus.cc M src/kudu/consensus/raft_consensus.h 4 files changed, 16 insertions(+), 17 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/89/17489/1 -- To view, visit http://gerrit.cloudera.org:8080/17489 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6a5082fe07449ef24827696638ddd8e925a08264 Gerrit-Change-Number: 17489 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
