Hello Kudu Jenkins, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9387
to look at the new patch set (#2).
Change subject: [consensus_peers] micro-optimization on controller status
......................................................................
[consensus_peers] micro-optimization on controller status
While running raft_consensus_stress-itest where a lot of RPC fail,
I found the following stats for operator new[](unsigned long)
in tserver with regard of Status::CopyState():
- 1.13% kudu-tserver libtcmalloc.so.4.5.1 [.] operator
new[](unsigned long)
- operator new[](unsigned long)
...
+ 3.21% kudu::Status::CopyState(char const*)
kudu::Status::Status(kudu::Status const&)
kudu::rpc::OutboundCall::status() const
kudu::rpc::RpcController::status() const
kudu::consensus::Peer::ProcessResponse()
...
Of course, that's not in the hot path in case of success scenarios,
but I think it makes sense to add these micro-optimizations.
Change-Id: I25ac21db191a21f3bdfb3378e32fbe366d98297c
---
M src/kudu/consensus/consensus_peers.cc
1 file changed, 9 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/9387/2
--
To view, visit http://gerrit.cloudera.org:8080/9387
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I25ac21db191a21f3bdfb3378e32fbe366d98297c
Gerrit-Change-Number: 9387
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
