Repository: incubator-impala Updated Branches: refs/heads/master 57210ecd5 -> cd6d86b83
IMPALA-1616: Improve the Memory Limit Exceeded error report The error report has been changed to include the id of the fragment instance that exceeded the memory limit. Change-Id: Ibb4e0c359d889938b4c351771ba539850bdb95ea Reviewed-on: http://gerrit.cloudera.org:8080/4335 Reviewed-by: Michael Ho <[email protected]> Tested-by: Internal Jenkins Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/3be61f90 Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/3be61f90 Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/3be61f90 Branch: refs/heads/master Commit: 3be61f902425a026a5f29f6943f9d236f77f58b4 Parents: 57210ec Author: Attila Jeges <[email protected]> Authored: Thu Sep 1 12:04:38 2016 -0700 Committer: Internal Jenkins <[email protected]> Committed: Mon Sep 19 23:39:33 2016 +0000 ---------------------------------------------------------------------- be/src/runtime/runtime-state.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/3be61f90/be/src/runtime/runtime-state.cc ---------------------------------------------------------------------- diff --git a/be/src/runtime/runtime-state.cc b/be/src/runtime/runtime-state.cc index 36e9bcf..57aae58 100644 --- a/be/src/runtime/runtime-state.cc +++ b/be/src/runtime/runtime-state.cc @@ -247,7 +247,7 @@ void RuntimeState::LogMemLimitExceeded(const MemTracker* tracker, DCHECK_GE(failed_allocation_size, 0); DCHECK(query_mem_tracker_.get() != NULL); stringstream ss; - ss << "Memory Limit Exceeded\n"; + ss << "Memory Limit Exceeded by fragment: " << fragment_instance_id() << endl; if (failed_allocation_size != 0) { DCHECK(tracker != NULL); ss << " " << tracker->label() << " could not allocate "
