IMPALA-4136: testKudu planner test hangs if Kudu is not supported The Kudu planner tests were recently moved into PlannerTest but they need to be explicitly skipped when KUDU_IS_SUPPORTED is false.
Change-Id: Iaf713107cdf49fa73efae5a731f26bbce96d8364 Reviewed-on: http://gerrit.cloudera.org:8080/4426 Reviewed-by: Alex Behm <[email protected]> Tested-by: Internal 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/0f188636 Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/0f188636 Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/0f188636 Branch: refs/heads/master Commit: 0f188636d029c19b06a23f60ba63b47f4cd2d866 Parents: 1e33d04 Author: Matthew Jacobs <[email protected]> Authored: Thu Sep 15 09:32:56 2016 -0700 Committer: Internal Jenkins <[email protected]> Committed: Fri Sep 16 20:37:55 2016 +0000 ---------------------------------------------------------------------- .../cloudera/impala/planner/PlannerTest.java | 23 ++++++++++++++++---- 1 file changed, 19 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/0f188636/fe/src/test/java/com/cloudera/impala/planner/PlannerTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/com/cloudera/impala/planner/PlannerTest.java b/fe/src/test/java/com/cloudera/impala/planner/PlannerTest.java index a09f218..6c88170 100644 --- a/fe/src/test/java/com/cloudera/impala/planner/PlannerTest.java +++ b/fe/src/test/java/com/cloudera/impala/planner/PlannerTest.java @@ -17,9 +17,11 @@ package com.cloudera.impala.planner; +import org.junit.Assume; import org.junit.Test; import com.cloudera.impala.catalog.Db; +import com.cloudera.impala.common.RuntimeEnv; import com.cloudera.impala.thrift.TExplainLevel; import com.cloudera.impala.thrift.TQueryOptions; import com.cloudera.impala.thrift.TRuntimeFilterMode; @@ -247,21 +249,34 @@ public class PlannerTest extends PlannerTestBase { } @Test - public void testKudu() { runPlannerTestFile("kudu"); } + public void testKudu() { + Assume.assumeTrue(RuntimeEnv.INSTANCE.isKuduSupported()); + runPlannerTestFile("kudu"); + } @Test - public void testKuduUpdate() { runPlannerTestFile("kudu-update"); } + public void testKuduUpdate() { + Assume.assumeTrue(RuntimeEnv.INSTANCE.isKuduSupported()); + runPlannerTestFile("kudu-update"); + } @Test - public void testKuduDelete() { runPlannerTestFile("kudu-delete"); } + public void testKuduDelete() { + Assume.assumeTrue(RuntimeEnv.INSTANCE.isKuduSupported()); + runPlannerTestFile("kudu-delete"); + } @Test public void testKuduSelectivity() { + Assume.assumeTrue(RuntimeEnv.INSTANCE.isKuduSupported()); TQueryOptions options = defaultQueryOptions(); options.setExplain_level(TExplainLevel.VERBOSE); runPlannerTestFile("kudu-selectivity", options); } @Test - public void testKuduTpch() { runPlannerTestFile("tpch-kudu"); } + public void testKuduTpch() { + Assume.assumeTrue(RuntimeEnv.INSTANCE.isKuduSupported()); + runPlannerTestFile("tpch-kudu"); + } }
