[impala] 04/04: IMPALA-8113: skip test_aggregation and test_avro_primitive_in_list in S3

[mi...@apache.org]

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

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

commit 678ed162d468be53faf443e5188b3d41fe5636fd
Author: Michael Brown <mi...@cloudera.com>
AuthorDate: Thu Jan 24 14:41:07 2019 -0800

    IMPALA-8113: skip test_aggregation and test_avro_primitive_in_list in S3
    
    Procedure for flaky S3 tests right now is to skip them due to eventual
    consistency problems. This skips 2 more, test_aggregation and
    test_avro_primitive_in_list.
    
    Change-Id: Ica001e0237cf1eff300c6b4e8f28f48e728a0649
    Reviewed-on: http://gerrit.cloudera.org:8080/12272
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/query_test/test_aggregation.py  | 2 ++
 tests/query_test/test_nested_types.py | 1 +
 2 files changed, 3 insertions(+)

diff --git a/tests/query_test/test_aggregation.py b/tests/query_test/test_aggregation.py
index 3b69c99..038790e 100644
--- a/tests/query_test/test_aggregation.py
+++ b/tests/query_test/test_aggregation.py
@@ -21,6 +21,7 @@ import pytest
 
 from testdata.common import widetable
 from tests.common.impala_test_suite import ImpalaTestSuite
+from tests.common.skip import SkipIfS3
 from tests.common.test_dimensions import (
     create_exec_option_dimension,
     create_exec_option_dimension_from_dict,
@@ -116,6 +117,7 @@ class TestAggregation(ImpalaTestSuite):
       return False
     return True
 
+  @SkipIfS3.eventually_consistent
   def test_aggregation(self, vector):
     exec_option = vector.get_value('exec_option')
     disable_codegen = exec_option['disable_codegen']
diff --git a/tests/query_test/test_nested_types.py b/tests/query_test/test_nested_types.py
index 899f504..afe4e56 100644
--- a/tests/query_test/test_nested_types.py
+++ b/tests/query_test/test_nested_types.py
@@ -273,6 +273,7 @@ class TestParquetArrayEncodings(ImpalaTestSuite):
   # .array = 34
   # .array = 35
   # .array = 36
+  @SkipIfS3.eventually_consistent
   def test_avro_primitive_in_list(self, vector, unique_database):
     self.__test_primitive_in_list(unique_database, "AvroPrimitiveInList",
       "AvroPrimitiveInList.parquet", vector)