[impala] branch master updated: IMPALA-10366: skip test_runtime_profile_aggregated for EC

[tarmstrong]

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

tarmstrong 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 62c19e6  IMPALA-10366: skip test_runtime_profile_aggregated for EC
62c19e6 is described below

commit 62c19e63396c1e783e93b4e7fb71804b70955e89
Author: Tim Armstrong <tarmstr...@cloudera.com>
AuthorDate: Mon Nov 30 19:53:49 2020 -0800

    IMPALA-10366: skip test_runtime_profile_aggregated for EC
    
    The schedule for erasure coded data results in 3 instead
    of 4 instances of the fragment with the scan. Skip the
    test - we don't need special coverage for erasure coding.
    
    Change-Id: I2bb47d89f6d6c59242f2632c481f26d93e28e33e
    Reviewed-on: http://gerrit.cloudera.org:8080/16799
    Reviewed-by: Aman Sinha <amsi...@cloudera.com>
    Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
 tests/common/skip.py                         | 1 +
 tests/custom_cluster/test_runtime_profile.py | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/tests/common/skip.py b/tests/common/skip.py
index a340360..21bbfdd 100644
--- a/tests/common/skip.py
+++ b/tests/common/skip.py
@@ -195,6 +195,7 @@ class SkipIfEC:
   fix_later = pytest.mark.skipif(IS_EC, reason="It should work but doesn't.")
   contain_full_explain = pytest.mark.skipif(IS_EC, reason="Contain full 
explain output "
               "for hdfs tables.")
+  different_schedule = pytest.mark.skipif(IS_EC, reason="Query is scheduled 
differently.")
 
 
 class SkipIfDockerizedCluster:
diff --git a/tests/custom_cluster/test_runtime_profile.py 
b/tests/custom_cluster/test_runtime_profile.py
index 73d7b50..933ad7c 100644
--- a/tests/custom_cluster/test_runtime_profile.py
+++ b/tests/custom_cluster/test_runtime_profile.py
@@ -17,6 +17,7 @@
 
 import pytest
 from tests.common.custom_cluster_test_suite import CustomClusterTestSuite
+from tests.common.skip import SkipIfEC
 
 
 class TestRuntimeProfile(CustomClusterTestSuite):
@@ -28,6 +29,7 @@ class TestRuntimeProfile(CustomClusterTestSuite):
 
   PERIODIC_COUNTER_UPDATE_FLAG = '--periodic_counter_update_period_ms=50'
 
+  @SkipIfEC.different_schedule
   @pytest.mark.execute_serially
   @CustomClusterTestSuite.with_args('--gen_experimental_profile=true ' +
       PERIODIC_COUNTER_UPDATE_FLAG)