Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24848 )
Change subject: IMPALA-15181: Smallify strings in GroupingAggregator's DeepCopy ...................................................................... IMPALA-15181: Smallify strings in GroupingAggregator's DeepCopy Small strings are now smallified during the deep copy of GroupingAggregator, and var len memory is not allocated for them. Testing: validated that small strings are smallified when executing TPC-DS Q67 using debug code. Change-Id: I22311d9cb678874d5c53f19811fa335b646c8d6f Reviewed-on: http://gerrit.cloudera.org:8080/24848 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/grouping-aggregator-ir.cc M be/src/exec/grouping-aggregator.cc M be/src/runtime/string-value.h 3 files changed, 5 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24848 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I22311d9cb678874d5c53f19811fa335b646c8d6f Gerrit-Change-Number: 24848 Gerrit-PatchSet: 4 Gerrit-Owner: Balazs Hevele <[email protected]> Gerrit-Reviewer: Balazs Hevele <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
