Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10989 )
Change subject: IMPALA-7256: Aggregator mem usage isn't reflected in summary ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/10989/1/be/src/exec/exec-node.cc File be/src/exec/exec-node.cc: http://gerrit.cloudera.org:8080/#/c/10989/1/be/src/exec/exec-node.cc@113 PS1, Line 113: expr_mem_tracker_.reset(new MemTracker(-1, "Exprs", mem_tracker_.get(), false)); > The parallel MemTracker/ReservationTracker hierarchy is pretty annoying her Done -- To view, visit http://gerrit.cloudera.org:8080/10989 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iba6ef207bed47810fc742aec3481db5f313cf97f Gerrit-Change-Number: 10989 Gerrit-PatchSet: 2 Gerrit-Owner: Thomas Marshall <thomasmarsh...@cmu.edu> Gerrit-Reviewer: Thomas Marshall <thomasmarsh...@cmu.edu> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Thu, 19 Jul 2018 23:38:54 +0000 Gerrit-HasComments: Yes