You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Internal Jenkins (Code Review)" <ge...@cloudera.org> on 2016/06/01 05:35:34 UTC

[Impala-CR](cdh5-trunk) IMPALA-3647: track runtime filter memory in separate tracker

Internal Jenkins has submitted this change and it was merged.

Change subject: IMPALA-3647: track runtime filter memory in separate tracker
......................................................................


IMPALA-3647: track runtime filter memory in separate tracker

This change breaks out runtime filter memory consumption from the
query-wide tracker to improve debuggability of memory limit exceeded
errors.

Testing: ran exhaustive tests, ran local and cluster stress tests.

Change-Id: I9f28f3b55b5c62e6f0f9838c5947c9446d444d20
Reviewed-on: http://gerrit.cloudera.org:8080/3247
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Reviewed-by: Michael Ho <kw...@cloudera.com>
Tested-by: Internal Jenkins
---
M be/src/runtime/plan-fragment-executor.cc
M be/src/runtime/runtime-filter-bank.cc
M be/src/runtime/runtime-filter-bank.h
M be/src/runtime/runtime-state.cc
M be/src/runtime/runtime-state.h
5 files changed, 23 insertions(+), 9 deletions(-)

Approvals:
  Michael Ho: Looks good to me, but someone else must approve
  Internal Jenkins: Verified
  Tim Armstrong: Looks good to me, approved



-- 
To view, visit http://gerrit.cloudera.org:8080/3247
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9f28f3b55b5c62e6f0f9838c5947c9446d444d20
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Michael Ho
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>