You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by cs...@apache.org on 2025/08/27 18:11:39 UTC

(impala) branch master updated: IMPALA-14171: Deflake TestQueryExecTimeLimit.test_exec_time_limit_long_plan

This is an automated email from the ASF dual-hosted git repository.

csringhofer pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new 669d74f46 IMPALA-14171: Deflake TestQueryExecTimeLimit.test_exec_time_limit_long_plan
669d74f46 is described below

commit 669d74f467193b575ec203ce45c5b3c02141a257
Author: stiga-huang <hu...@gmail.com>
AuthorDate: Wed Aug 27 10:03:49 2025 +0800

    IMPALA-14171: Deflake TestQueryExecTimeLimit.test_exec_time_limit_long_plan
    
    The test expects a "SELECT..LIMIT 1" query on tpch.lineitem to finish in
    2s. This could be impacted by other concurrent tests when memory
    reservation is used up. This patch marks the test to run serially to
    avoid the impact from other tests.
    
    Change-Id: Ibbb2f1a34e24c83a3d2c69d2daa4dece8d94ec1e
    Reviewed-on: http://gerrit.cloudera.org:8080/23351
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/query_test/test_exec_time_limit.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tests/query_test/test_exec_time_limit.py b/tests/query_test/test_exec_time_limit.py
index 1806db054..589a1f424 100644
--- a/tests/query_test/test_exec_time_limit.py
+++ b/tests/query_test/test_exec_time_limit.py
@@ -17,6 +17,7 @@
 
 from __future__ import absolute_import
 from tests.common.impala_test_suite import ImpalaTestSuite
+import pytest
 
 
 class TestQueryExecTimeLimit(ImpalaTestSuite):
@@ -39,9 +40,11 @@ class TestQueryExecTimeLimit(ImpalaTestSuite):
         "Unexpected exception: {}".format(e)
       )
 
+  @pytest.mark.execute_serially
   def test_exec_time_limit_long_plan(self):
     """Test that queries with a long planning time completing within
-    exec_time_limit_s succeed."""
+    exec_time_limit_s succeed. Run this test serially to avoid impact from other
+    concurrent tests."""
     exec_options = dict()
     exec_options['exec_time_limit_s'] = "2"
     # Set debug action to wait in the plan phase for 10s.