[impala] branch master updated: IMPALA-11476: (Addendum) enable test_erasure_coding for Ozone

[mi...@apache.org]

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

michaelsmith 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 5018a30d6 IMPALA-11476: (Addendum) enable test_erasure_coding for Ozone
5018a30d6 is described below

commit 5018a30d6d4cd26ded8c68da965bbf526d0cb0e2
Author: Michael Smith <mi...@cloudera.com>
AuthorDate: Fri Mar 3 14:20:14 2023 -0800

    IMPALA-11476: (Addendum) enable test_erasure_coding for Ozone
    
    Enables test_erasure_coding for Ozone. HDDS-7603 is planned for Ozone
    1.4.0, but we test with a CDP build - 1.3.0.7.2.17.0-127 - that already
    includes this fix. Since this is a testing-only change, seems safe to
    rely on that.
    
    Testing:
    - Ran test_erasure_coding on Ozone with EC
    
    Change-Id: Iee57c008102db7fac89abcea9a140c867178bb08
    Reviewed-on: http://gerrit.cloudera.org:8080/19578
    Reviewed-by: Laszlo Gaal <la...@cloudera.com>
    Tested-by: Michael Smith <mi...@cloudera.com>
---
 tests/common/skip.py              | 2 --
 tests/query_test/test_scanners.py | 1 -
 2 files changed, 3 deletions(-)

diff --git a/tests/common/skip.py b/tests/common/skip.py
index c79f80ca9..48d03946d 100644
--- a/tests/common/skip.py
+++ b/tests/common/skip.py
@@ -115,8 +115,6 @@ class SkipIf:
   is_buggy_el6_kernel = pytest.mark.skipif(
       IS_BUGGY_EL6_KERNEL, reason="Kernel is affected by KUDU-1508")
 
-  ozone_ec_incomplete = pytest.mark.skipif(IS_OZONE and IS_EC, reason="HDDS-7603")
-
 class SkipIfLocal:
   # These are skipped due to product limitations.
   hdfs_blocks = pytest.mark.skipif(IS_LOCAL,
diff --git a/tests/query_test/test_scanners.py b/tests/query_test/test_scanners.py
index 5f3ae18f9..1813483fa 100644
--- a/tests/query_test/test_scanners.py
+++ b/tests/query_test/test_scanners.py
@@ -1896,7 +1896,6 @@ class TestErasureCoding(ImpalaTestSuite):
   def get_workload(cls):
     return 'functional-query'
 
-  @SkipIf.ozone_ec_incomplete
   @SkipIf.not_ec
   def test_erasure_coding(self, vector):
     self.run_test_case('QueryTest/hdfs-erasure-coding', vector)