You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2016/09/16 21:37:18 UTC

[kudu-CR] [c++client] performance optimizations

Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/4385

to look at the new patch set (#6).

Change subject: [c++client] performance optimizations
......................................................................

[c++client] performance optimizations

The change on Batcher::ComputeDeadlineUnlocked() gave
about 50% boost while running scenarios when timeout for session
operations is not set and small raw (wire) operation size (~100 bytes).
Also, avoid calling std::shared_from_this(KuduSession)
for every scheduled write operation.

Change-Id: I4b57fc7355f9f673f30861ec30cb6b48cdf656d2
---
M src/kudu/client/batcher.cc
M src/kudu/client/batcher.h
M src/kudu/client/client.cc
M src/kudu/client/session-internal.cc
M src/kudu/client/session-internal.h
5 files changed, 38 insertions(+), 43 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/4385/6
-- 
To view, visit http://gerrit.cloudera.org:8080/4385
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4b57fc7355f9f673f30861ec30cb6b48cdf656d2
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>