[impala] 06/08: IMPALA-8585: Skip most ACID EE tests in non-HDFS builds

[jo...@apache.org]

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

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

commit f6b72aa24add8232e1a20ac291c4400395839c74
Author: Csaba Ringhofer <cs...@cloudera.com>
AuthorDate: Thu Jul 25 21:10:24 2019 +0200

    IMPALA-8585: Skip most ACID EE tests in non-HDFS builds
    
    Currently these tests rely on Hive to insert data into ACID tables
    (+ also compaction in some cases), which fails on non-HDFS filesystems
    (e.g. S3).
    
    Change-Id: I5f67654805b770d86090b0d1015e144147d73f40
    Reviewed-on: http://gerrit.cloudera.org:8080/13921
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/query_test/test_acid.py | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/tests/query_test/test_acid.py b/tests/query_test/test_acid.py
index d39ba9f..208315d 100644
--- a/tests/query_test/test_acid.py
+++ b/tests/query_test/test_acid.py
@@ -17,7 +17,8 @@
 
 # Functional tests for ACID integration with Hive.
 from tests.common.impala_test_suite import ImpalaTestSuite
-from tests.common.skip import (SkipIfHive2, SkipIfCatalogV2)
+from tests.common.skip import (SkipIfHive2, SkipIfCatalogV2, SkipIfS3, SkipIfABFS,
+                               SkipIfADLS, SkipIfIsilon, SkipIfLocal)
 from tests.common.test_dimensions import create_single_exec_option_dimension
 
 
@@ -36,10 +37,20 @@ class TestAcid(ImpalaTestSuite):
         v.get_value('table_format').file_format in ['text'])
 
   @SkipIfHive2.acid
+  @SkipIfS3.hive
+  @SkipIfABFS.hive
+  @SkipIfADLS.hive
+  @SkipIfIsilon.hive
+  @SkipIfLocal.hive
   def test_acid(self, vector, unique_database):
     self.run_test_case('QueryTest/acid', vector, use_db=unique_database)
 
   @SkipIfHive2.acid
+  @SkipIfS3.hive
+  @SkipIfABFS.hive
+  @SkipIfADLS.hive
+  @SkipIfIsilon.hive
+  @SkipIfLocal.hive
   def test_acid_compaction(self, vector, unique_database):
     self.run_test_case('QueryTest/acid-compaction', vector, use_db=unique_database)
 
@@ -48,6 +59,11 @@ class TestAcid(ImpalaTestSuite):
     self.run_test_case('QueryTest/acid-negative', vector, use_db=unique_database)
 
   @SkipIfHive2.acid
+  @SkipIfS3.hive
+  @SkipIfABFS.hive
+  @SkipIfADLS.hive
+  @SkipIfIsilon.hive
+  @SkipIfLocal.hive
   def test_acid_partitioned(self, vector, unique_database):
     self.run_test_case('QueryTest/acid-partitioned', vector, use_db=unique_database)
 
@@ -57,6 +73,11 @@ class TestAcid(ImpalaTestSuite):
   # it can not be shown in the query profile.  Skip CatalogV2 to avoid flaky tests.
   @SkipIfHive2.acid
   @SkipIfCatalogV2.hms_event_polling_enabled()
+  @SkipIfS3.hive
+  @SkipIfABFS.hive
+  @SkipIfADLS.hive
+  @SkipIfIsilon.hive
+  @SkipIfLocal.hive
   def test_acid_profile(self, vector, unique_database):
     self.run_test_case('QueryTest/acid-profile', vector, use_db=unique_database)
 # TODO(todd): further tests to write: