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 (#7). 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, 147 insertions(+), 18 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/10731/7 -- 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: 7 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>