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]>

Reply via email to