Riza Suminto has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/23072


Change subject: IMPALA-14163: Raise test_pool_config_change_while_queued 
MEM_LIMIT
......................................................................

IMPALA-14163: Raise test_pool_config_change_while_queued MEM_LIMIT

test_pool_config_change_while_queued hit timeout at
TestAdmissionControllerWithACService. When running this test locally, we
notice that some trigger query ("select 'wait_for_config_change'")
passed when it is expected to be rejected (hit EXCEPTION during
admission).

This patch increase the MEM_LIMIT higher to 128GB to ensure rejection.
It also add wait_for_admission_control() that should immediately return
once trigger query hit exception. Removed redundant
"set enable_trivial_query_for_admission=false" query in
test_pool_config_change_while_queued.

Testing:
- Loop the test couple times and confirm that all trigger query
  executions hit exception.

Change-Id: Iee808d0fc92308604ed0ee27dde795e9aa69eb5d
---
M tests/common/resource_pool_config.py
M tests/custom_cluster/test_admission_controller.py
2 files changed, 3 insertions(+), 3 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee808d0fc92308604ed0ee27dde795e9aa69eb5d
Gerrit-Change-Number: 23072
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>

Reply via email to