Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7597/1/be/src/runtime/mem-tracker.cc File be/src/runtime/mem-tracker.cc: PS1, Line 344: 1 > What do you think about logging 2 levels down to get the query MemTrackers? +1 I was wondering about this as well. The pool tracker is probably not going to be particularly helpful when you hit the proc mem limit and want to know who the offender was. -- To view, visit http://gerrit.cloudera.org:8080/7597 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib151c3894d4c43b508779e6809d77e2f8af89cf2 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: Yes
