You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Sailesh Mukil (Code Review)" <ge...@cloudera.org> on 2017/04/06 22:50:02 UTC

[kudu-CR] WIP: KUDU-1865 (part 2): pre-allocate OutboundTransfer for inbound calls

Hello Kudu Jenkins,

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

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

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

Change subject: WIP: KUDU-1865 (part 2): pre-allocate OutboundTransfer for inbound calls
......................................................................

WIP: KUDU-1865 (part 2): pre-allocate OutboundTransfer for inbound calls

This allocates the OutboundTransfer object on the reactor when a call is
first received, to avoid a cross-thread alloc-free pair per RPC.

Conflicts:
	src/kudu/rpc/inbound_call.cc
	src/kudu/rpc/transfer.h

Change-Id: If2ce44d0a00e2dafd1a9858d758ea83ac7923fc0
---
M src/kudu/rpc/connection.cc
M src/kudu/rpc/inbound_call.cc
M src/kudu/rpc/inbound_call.h
M src/kudu/rpc/transfer.cc
M src/kudu/rpc/transfer.h
5 files changed, 56 insertions(+), 30 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/5906/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5906
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If2ce44d0a00e2dafd1a9858d758ea83ac7923fc0
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>