You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Pooja Nilangekar (Code Review)" <ge...@cloudera.org> on 2018/10/15 16:57:42 UTC

[Impala-ASF-CR] IMPALA-7545: Add queuing reason to query log

Pooja Nilangekar has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/11669 )

Change subject: IMPALA-7545: Add queuing reason to query log
......................................................................

IMPALA-7545: Add queuing reason to query log

After this change, the HS2 GetLog() function returns the queuing
reason for a query when it is queued by the AdmissionController.

Testing: Added an end-to-end test to test_admission_controller.py
to verify the query logs returned.

Change-Id: I2e5d8de4f6691a9ba2594ca68c54ea4dca760545
---
M be/src/scheduling/admission-controller.cc
M be/src/scheduling/admission-controller.h
M be/src/service/impala-hs2-server.cc
M tests/custom_cluster/test_admission_controller.py
4 files changed, 89 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/69/11669/5
-- 
To view, visit http://gerrit.cloudera.org:8080/11669
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2e5d8de4f6691a9ba2594ca68c54ea4dca760545
Gerrit-Change-Number: 11669
Gerrit-PatchSet: 5
Gerrit-Owner: Pooja Nilangekar <po...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Pooja Nilangekar <po...@cloudera.com>