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 2020/04/14 23:01:53 UTC

[impala] 01/03: IMPALA-9071: remove xfail from ACID CTAS test

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 82ca3262b3aeee13d2cc27c5ff9aef3f6c70cc8a
Author: Csaba Ringhofer <cs...@cloudera.com>
AuthorDate: Tue Apr 14 17:21:30 2020 +0200

    IMPALA-9071: remove xfail from ACID CTAS test
    
    The test passes at the moment - according to the TODO
    it needed HIVE-22158, which is included in CDP dependencies
    since a long time.
    
    Change-Id: I4e068879100061bb7e724f0bd1e5e4aa8edd7825
    Reviewed-on: http://gerrit.cloudera.org:8080/15729
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/custom_cluster/test_custom_hive_configs.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tests/custom_cluster/test_custom_hive_configs.py b/tests/custom_cluster/test_custom_hive_configs.py
index 513c86a..933a114 100644
--- a/tests/custom_cluster/test_custom_hive_configs.py
+++ b/tests/custom_cluster/test_custom_hive_configs.py
@@ -33,9 +33,7 @@ class TestCustomHiveConfigs(CustomClusterTestSuite):
   def setup_class(cls):
     super(TestCustomHiveConfigs, cls).setup_class()
 
-  # TODO: Remove the xfail marker after bumping CDP_BUILD_NUMBER to contain HIVE-22158
   @SkipIfHive2.acid
-  @pytest.mark.xfail(run=True, reason="May fail on Hive3 versions without HIVE-22158")
   @pytest.mark.execute_serially
   @CustomClusterTestSuite.with_args(hive_conf_dir=HIVE_SITE_EXT_DIR)
   def test_ctas_read_write_consistence(self, unique_database):