Zoltan Borok-Nagy has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21508


Change subject: IMPALA-13155: Smallify strings values in all Tuple::Deepcopy()
......................................................................

IMPALA-13155: Smallify strings values in all Tuple::Deepcopy()

Currently "Tuple::DeepCopy(const TupleDescriptor& desc, char** data,
int* offset, bool convert_ptrs)" does not try to smallify string values,
although it could safely do that as it creates new string values. We use
that version of DeepCopy when we BROADCAST data between fragments, so
smallifying on that path can be beneficial.

TODO: measurements

Change-Id: I5822243629e198e56006d81a9e5077926f391a74
---
M be/src/runtime/tuple.cc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/08/21508/1
--
To view, visit http://gerrit.cloudera.org:8080/21508
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5822243629e198e56006d81a9e5077926f391a74
Gerrit-Change-Number: 21508
Gerrit-PatchSet: 1
Gerrit-Owner: Zoltan Borok-Nagy <borokna...@cloudera.com>

Reply via email to