Hello Bikramjeet Vig, Dan Hecht, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10731
to look at the new patch set (#8).
Change subject: IMPALA-6969: add AC last queued reason to profile
......................................................................
IMPALA-6969: add AC last queued reason to profile
The reason is updated during initial admission and when the query is at
the head of the queue but can't be admitted. It is not updated while
the query is in the middle of the queue.
Together with the async admission change, this makes it possible to
determine from the profile why the query has not been admitted yet.
Testing:
Added admission control tests that check that the
string is set for queries queued based both on the
query count and the max memory.
Looped the tests overnight to confirm non-flakiness.
Change-Id: Ida9b75dc50dfb7a27f59deda91bad6ac838130a1
---
M be/src/scheduling/admission-controller.cc
M be/src/scheduling/admission-controller.h
M tests/custom_cluster/test_admission_controller.py
3 files changed, 148 insertions(+), 18 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/10731/8
--
To view, visit http://gerrit.cloudera.org:8080/10731
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ida9b75dc50dfb7a27f59deda91bad6ac838130a1
Gerrit-Change-Number: 10731
Gerrit-PatchSet: 8
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
