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 2022/04/13 09:46:59 UTC

[impala] 03/03: IMPALA-11241: Mark test_statement_expression_limit as serialized

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

commit 011bef6249bd9b0313f3602105f58fa6d239d5bb
Author: Csaba Ringhofer <cs...@cloudera.com>
AuthorDate: Wed Apr 13 10:44:13 2022 +0200

    IMPALA-11241: Mark test_statement_expression_limit as serialized
    
    This test takes ~2GB of JVM heap and is a likely cause of some
    OOM during EE tests.
    
    Change-Id: I4868eab51bf16e1ca6e0b4e98d7929195cd73803
    Reviewed-on: http://gerrit.cloudera.org:8080/18411
    Reviewed-by: Quanlong Huang <hu...@gmail.com>
    Tested-by: Csaba Ringhofer <cs...@cloudera.com>
---
 tests/query_test/test_exprs.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/query_test/test_exprs.py b/tests/query_test/test_exprs.py
index 263942b2b..cf98fc7c8 100644
--- a/tests/query_test/test_exprs.py
+++ b/tests/query_test/test_exprs.py
@@ -164,6 +164,9 @@ class TestExprLimits(ImpalaTestSuite):
     err = self.execute_query_expect_failure(self.client, invalid_sql)
     assert re.search(expected_err_tmpl.format(len(invalid_sql), size_16mb), str(err))
 
+  # This test can take ~2GB memory while it takes only ~10 seconds. It caused OOM
+  # in the past, so it is safer to run it serially.
+  @pytest.mark.execute_serially
   def test_statement_expression_limit(self):
     """Generate a huge case statement that barely fits within the 16MB limit but exceeds
        the statement expression limit. Verify that it fails."""