You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by kw...@apache.org on 2017/04/26 23:22:16 UTC
[04/10] incubator-impala git commit: IMPALA-5244
test_hdfs_file_open_fail fails on local filesystem build
IMPALA-5244 test_hdfs_file_open_fail fails on local filesystem build
This test had to be skipped for non HDFS filesystems.
Change-Id: I5318a5eb27b15fed5df770b9c3ea23e7e1a97a4c
Reviewed-on: http://gerrit.cloudera.org:8080/6723
Reviewed-by: Michael Ho <kw...@cloudera.com>
Reviewed-by: Sailesh Mukil <sa...@cloudera.com>
Tested-by: Impala Public Jenkins
Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/edcc593e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/edcc593e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/edcc593e
Branch: refs/heads/master
Commit: edcc593ee541bb2677d58e31fe713f2cb4951937
Parents: 915a163
Author: Sailesh Mukil <sa...@cloudera.com>
Authored: Mon Apr 24 23:22:37 2017 -0700
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Tue Apr 25 10:50:18 2017 +0000
----------------------------------------------------------------------
tests/common/skip.py | 1 +
tests/data_errors/test_data_errors.py | 5 +++--
2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/edcc593e/tests/common/skip.py
----------------------------------------------------------------------
diff --git a/tests/common/skip.py b/tests/common/skip.py
index 232eab5..e330712 100644
--- a/tests/common/skip.py
+++ b/tests/common/skip.py
@@ -61,6 +61,7 @@ class SkipIf:
kudu_not_supported = pytest.mark.skipif(os.environ["KUDU_IS_SUPPORTED"] == "false",
reason="Kudu is not supported")
not_s3 = pytest.mark.skipif(not IS_S3, reason="S3 Filesystem needed")
+ not_hdfs = pytest.mark.skipif(not IS_HDFS, reason="HDFS Filesystem needed")
no_secondary_fs = pytest.mark.skipif(not SECONDARY_FILESYSTEM,
reason="Secondary filesystem needed")
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/edcc593e/tests/data_errors/test_data_errors.py
----------------------------------------------------------------------
diff --git a/tests/data_errors/test_data_errors.py b/tests/data_errors/test_data_errors.py
index 92ae707..60d70b7 100644
--- a/tests/data_errors/test_data_errors.py
+++ b/tests/data_errors/test_data_errors.py
@@ -24,7 +24,7 @@ import random
from tests.beeswax.impala_beeswax import ImpalaBeeswaxException
from tests.common.impala_test_suite import ImpalaTestSuite
-from tests.common.skip import SkipIfS3, SkipIfLocal
+from tests.common.skip import SkipIf, SkipIfS3, SkipIfLocal
from tests.common.test_dimensions import create_exec_option_dimension
class TestDataErrors(ImpalaTestSuite):
@@ -42,7 +42,8 @@ class TestDataErrors(ImpalaTestSuite):
def get_workload(self):
return 'functional-query'
-# Regression test for IMP-633. Added as a part of IMPALA-5198
+# Regression test for IMP-633. Added as a part of IMPALA-5198.
+@SkipIf.not_hdfs
class TestHdfsFileOpenFailErrors(ImpalaTestSuite):
@pytest.mark.execute_serially
def test_hdfs_file_open_fail(self):