Hello Thomas Marshall, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11203
to look at the new patch set (#2).
Change subject: Analytic mem mgmt cleanup
......................................................................
Analytic mem mgmt cleanup
Following on from IMPALA-7403, clean up some of memory management
in AnalyticEvalNode.
Remove an unnecessary MemPool and avoid reallocating tuples
unnecessarily in Reset(). Also switch to inline initialization
of members.
Testing:
Ran exhaustive tests
Change-Id: Id16beb2d0b9d4315f52dd45203649252c0eb06b0
---
M be/src/exec/analytic-eval-node.cc
M be/src/exec/analytic-eval-node.h
2 files changed, 54 insertions(+), 65 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/03/11203/2
--
To view, visit http://gerrit.cloudera.org:8080/11203
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id16beb2d0b9d4315f52dd45203649252c0eb06b0
Gerrit-Change-Number: 11203
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Thomas Marshall <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>