This is an automated email from the ASF dual-hosted git repository. asherman pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 8416432cc3e656b6c5fa230d4075ba64e0f3d091 Author: Andrew Sherman <[email protected]> AuthorDate: Mon Nov 25 17:28:20 2024 -0800 IMPALA-13560: Second attempt at fixing test_admission_controller_with_quota_configs test. The test test_admission_controller_with_quota_configs() is designed to be a similar test to test_admission_controller_with_configs() which uses the 'queueB' queue. The newer test uses a newly added queue 'queueF'. Because Admission Control configuration is split across two files, and because of user stupidity, the queue timeout configuration for 'queueB' was not copied when the new test was created. This causes queued queries to be timed out while waiting for admission, which confuses the test. Set pool-queue-timeout-ms.root to 600000 for queueF in llama-site-test2.xml. Change-Id: I1378cd4e42ed5629b92b1c16dd17d4d16ec4a19d Reviewed-on: http://gerrit.cloudera.org:8080/22126 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- fe/src/test/resources/llama-site-test2.xml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/fe/src/test/resources/llama-site-test2.xml b/fe/src/test/resources/llama-site-test2.xml index 045b72828..f7558872a 100644 --- a/fe/src/test/resources/llama-site-test2.xml +++ b/fe/src/test/resources/llama-site-test2.xml @@ -77,4 +77,8 @@ <name>llama.am.throttling.maximum.placed.reservations.root.queueD</name> <value>6</value> </property> + <property> + <name>impala.admission-control.pool-queue-timeout-ms.root.queueF</name> + <value>600000</value> + </property> </configuration>
