Internal Jenkins has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/4049 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Internal Jenkins --- 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(-) Approvals: Internal Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4049 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5ec5572b0e26898da352b7e6b11eb01c6edb2e5 Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
