Wenzhe Zhou has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19688


Change subject: IMPALA-12036: Fix Web UI to show right resource pools
......................................................................

IMPALA-12036: Fix Web UI to show right resource pools

Web UI always shows default resource pool even if a different resource
pool is used by the query. The Planner could set request_pool in
TQueryCtx, but the backend does not check the request_pool in TQueryCtx
returned from the frontend when getting resource pools for Web UI.
This patch fixs the issue in ClientRequestState::request_pool(). It also
removes the error path in RequestPoolService::ResolveRequestPool() if
requested_pool is empty string.

Testing:
 - Updated TestExecutorGroups::test_query_cpu_count_divisor_default to
   verify resource pools in Web queries site and Web admission site.

Change-Id: Iceacb3a8ec3bd15a8029ba05d064bbbb81e3a766
---
M be/src/scheduling/request-pool-service.cc
M be/src/service/client-request-state.h
M tests/custom_cluster/test_executor_groups.py
M tests/custom_cluster/test_web_pages.py
4 files changed, 27 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/88/19688/1
--
To view, visit http://gerrit.cloudera.org:8080/19688
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iceacb3a8ec3bd15a8029ba05d064bbbb81e3a766
Gerrit-Change-Number: 19688
Gerrit-PatchSet: 1
Gerrit-Owner: Wenzhe Zhou <[email protected]>

Reply via email to