You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by mj...@apache.org on 2016/09/16 21:44:08 UTC
[2/2] incubator-impala git commit: IMPALA-4136: testKudu planner test
hangs if Kudu is not supported
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 <al...@cloudera.com>
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 <mj...@cloudera.com>
Authored: Thu Sep 15 09:32:56 2016 -0700
Committer: Internal Jenkins <cl...@gerrit.cloudera.org>
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");
+ }
}