You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by as...@apache.org on 2020/04/21 12:47:12 UTC

[impala] branch master updated: IMPALA-9665: Fixed database not found errors in query_test.test_insert

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


The following commit(s) were added to refs/heads/master by this push:
     new 02d84dc  IMPALA-9665: Fixed database not found errors in query_test.test_insert
02d84dc is described below

commit 02d84dcf502afbb610eb1f911b3a832adfbf5fa9
Author: Adam Tamas <ta...@cloudera.com>
AuthorDate: Mon Apr 20 23:51:29 2020 +0200

    IMPALA-9665: Fixed database not found errors in query_test.test_insert
    
    Fixed the usage of the unique_database in the test_insert.py  to wait with the
    tests until the database is synced.
    
    Testing:
    -tests/run-tests.py query_test/test_insert.py --exploration_strategy=exhaustive
    
    Change-Id: I9b7aa3775dd4375f536d76f2e236ce126f8c78cd
    Reviewed-on: http://gerrit.cloudera.org:8080/15766
    Reviewed-by: Andrew Sherman <as...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/query_test/test_insert.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/query_test/test_insert.py b/tests/query_test/test_insert.py
index 3223377..d9a7255 100644
--- a/tests/query_test/test_insert.py
+++ b/tests/query_test/test_insert.py
@@ -124,6 +124,7 @@ class TestInsertQueries(ImpalaTestSuite):
   def setup_class(cls):
     super(TestInsertQueries, cls).setup_class()
 
+  @UniqueDatabase.parametrize(sync_ddl=True)
   # Erasure coding doesn't respect memory limit
   @SkipIfEC.fix_later
   # ABFS partition names cannot end in periods
@@ -158,6 +159,7 @@ class TestInsertQueries(ImpalaTestSuite):
     self.run_test_case('QueryTest/acid-insert-fail', vector, unique_database,
         multiple_impalad=vector.get_value('exec_option')['sync_ddl'] == 1)
 
+  @UniqueDatabase.parametrize(sync_ddl=True)
   @pytest.mark.execute_serially
   @SkipIfNotHdfsMinicluster.tuned_for_minicluster
   def test_insert_mem_limit(self, vector, unique_database):
@@ -174,11 +176,13 @@ class TestInsertQueries(ImpalaTestSuite):
     for v in verifiers:
       v.wait_for_metric("impala-server.num-fragments-in-flight", 0, timeout=180)
 
+  @UniqueDatabase.parametrize(sync_ddl=True)
   @SkipIfS3.eventually_consistent
   def test_insert_overwrite(self, vector, unique_database):
     self.run_test_case('QueryTest/insert_overwrite', vector, unique_database,
         multiple_impalad=vector.get_value('exec_option')['sync_ddl'] == 1)
 
+  @UniqueDatabase.parametrize(sync_ddl=True)
   def test_insert_bad_expr(self, vector, unique_database):
     # The test currently relies on codegen being disabled to trigger an error in
     # the output expression of the table sink.