Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/24163
Change subject: KUDU-3758 avoid TransferPayload copying for
CreateForCall{Request,Response}
......................................................................
KUDU-3758 avoid TransferPayload copying for CreateForCall{Request,Response}
Even if copying of Slice objects is shallow (the underlying memory isn't
copied, just the pointer and the size are copied), it makes sense to
avoid heap memory allocation and copying of Slice objects themselves.
The latter happens when there are more than two side-cars in per RPC, so
TransferPayload (boost::container::small_vector<Slice, 4> typedef)
starts allocating its elements on the heap.
Change-Id: Ia26e1a18fbb903a4df2ab7b20549fca3c2b06ad2
Reviewed-on: http://gerrit.cloudera.org:8080/24160
Reviewed-by: Michael Smith <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
Tested-by: Alexey Serbin <[email protected]>
(cherry picked from commit 80c1b1dcb5d065f739fd74b0177f1f72e09733d9)
---
M src/kudu/rpc/connection.cc
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/24163/1
--
To view, visit http://gerrit.cloudera.org:8080/24163
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia26e1a18fbb903a4df2ab7b20549fca3c2b06ad2
Gerrit-Change-Number: 24163
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
