Alexey Serbin has submitted this change and it was merged. Change subject: [c++client] performance optimizations ......................................................................
[c++client] performance optimizations The change on Batcher::ComputeDeadlineUnlocked() gave about 50% boost for scenarios when session timeout is not set and write operations are small (raw/wire size ~100 bytes). Avoid calling std::shared_from_this(KuduSession) for every scheduled write operation. Change-Id: I4b57fc7355f9f673f30861ec30cb6b48cdf656d2 Reviewed-on: http://gerrit.cloudera.org:8080/4385 Tested-by: Kudu Jenkins Reviewed-by: David Ribeiro Alves <dral...@apache.org> --- 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, 45 insertions(+), 51 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4385 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b57fc7355f9f673f30861ec30cb6b48cdf656d2 Gerrit-PatchSet: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
