Tim Armstrong has uploaded a new patch set (#2).

Change subject: IMPALA-3090: always log memory limit errors
......................................................................

IMPALA-3090: always log memory limit errors

Consistently log memory limit errors so that the error message contains
a dump of the query memory trackers at the time that the memory limit
was hit (instead of after the fact when the query is already partially
cleaned up).

Testing:
Exhaustive build passed. Ran local stress test for a bit.

Change-Id: If5ec5572b0e26898da352b7e6b11eb01c6edb2e5
---
M be/src/exec/analytic-eval-node.cc
M be/src/exec/partitioned-hash-join-node.cc
M be/src/runtime/buffered-block-mgr.cc
M be/src/runtime/mem-tracker.h
M be/src/runtime/plan-fragment-executor.cc
M be/src/runtime/row-batch.cc
M be/src/runtime/runtime-state.h
M be/src/runtime/sorter.cc
8 files changed, 38 insertions(+), 45 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/4049/2
-- 
To view, visit http://gerrit.cloudera.org:8080/4049
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If5ec5572b0e26898da352b7e6b11eb01c6edb2e5
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Michael Ho
Gerrit-Reviewer: Michael Ho <[email protected]>

Reply via email to