(impala) 03/03: IMPALA-13560: Second attempt at fixing test_admission_controller_with_quota_configs test.

[as...@apache.org]

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 <as...@cloudera.com>
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 <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 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>