You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by st...@apache.org on 2022/04/02 05:43:42 UTC

[impala] 01/02: IMPALA-111218: TestIcebergTable.test_table_load_time_for_many_files flaky

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

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

commit 84d67f09b4864f292ca56dc05464e7a6a16bc0f4
Author: Zoltan Borok-Nagy <bo...@cloudera.com>
AuthorDate: Fri Apr 1 21:04:27 2022 +0200

    IMPALA-111218: TestIcebergTable.test_table_load_time_for_many_files flaky
    
    In some test runs test_table_load_time_for_many_files failed because
    the time limit (3 seconds) was too short. In the failed tests it was
    around 4 seconds to execute the DESCRIBE statement.
    
    This patch increases the time limit to 10 seconds when HDFS is used.
    This should be long enough to fix the flakiness, but short enough
    to catch pathologically long table loads (the one that was fixed by
    IMPALA-11214).
    
    Change-Id: Ic48bb561e97c008c6f4e51fc9d9ec8b3f7cdaf9e
    Reviewed-on: http://gerrit.cloudera.org:8080/18376
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/query_test/test_iceberg.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/query_test/test_iceberg.py b/tests/query_test/test_iceberg.py
index 23b934f..322b7c9 100644
--- a/tests/query_test/test_iceberg.py
+++ b/tests/query_test/test_iceberg.py
@@ -563,9 +563,9 @@ class TestIcebergTable(ImpalaTestSuite):
     self.execute_query("describe formatted {}".format(tbl_name))
     elapsed_time = time.time() - start_time
     if IS_HDFS:
-      time_limit = 3
-    else:
       time_limit = 10
+    else:
+      time_limit = 20
     assert elapsed_time < time_limit
 
   def test_consistent_scheduling(self, vector, unique_database):