You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by sp...@apache.org on 2016/05/26 15:40:44 UTC

[22/66] [abbrv] hive git commit: HIVE-13794: HIVE_RPC_QUERY_PLAN should always be set when generating LLAP splits (Jason Dere, reviewed by Siddharth Seth)

HIVE-13794: HIVE_RPC_QUERY_PLAN should always be set when generating LLAP splits (Jason Dere, reviewed by Siddharth Seth)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/2d103982
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/2d103982
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/2d103982

Branch: refs/heads/java8
Commit: 2d1039829a4a6b0978e423fbefabce22568ef8a2
Parents: 115d225
Author: Jason Dere <jd...@hortonworks.com>
Authored: Tue May 24 17:07:05 2016 -0700
Committer: Jason Dere <jd...@hortonworks.com>
Committed: Tue May 24 17:07:05 2016 -0700

----------------------------------------------------------------------
 .../src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java  | 3 ---
 .../apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java   | 2 ++
 2 files changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/2d103982/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
----------------------------------------------------------------------
diff --git a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
index 48b9493..5d0da30 100644
--- a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
+++ b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
@@ -103,9 +103,6 @@ public class TestJdbcWithMiniLlap {
 
     conf = new HiveConf();
     conf.setBoolVar(ConfVars.HIVE_SUPPORT_CONCURRENCY, false);
-    // Necessary for GetSplits()/LlapInputFormat,
-    // the config generated for the query fragment needs to include the MapWork
-    conf.setBoolVar(HiveConf.ConfVars.HIVE_RPC_QUERY_PLAN, true);
 
     conf.addResource(new URL("file://" + new File(confDir).toURI().getPath()
         + "/tez-site.xml"));

http://git-wip-us.apache.org/repos/asf/hive/blob/2d103982/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java b/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java
index ce69ee6..83d492a 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDTFGetSplits.java
@@ -205,6 +205,8 @@ public class GenericUDTFGetSplits extends GenericUDTF {
     HiveConf.setVar(conf, HiveConf.ConfVars.HIVE_EXECUTION_MODE, "llap");
     HiveConf.setBoolVar(conf, HiveConf.ConfVars.HIVE_TEZ_GENERATE_CONSISTENT_SPLITS, true);
     HiveConf.setBoolVar(conf, HiveConf.ConfVars.LLAP_CLIENT_CONSISTENT_SPLITS, true);
+    // Tez/LLAP requires RPC query plan
+    HiveConf.setBoolVar(conf, HiveConf.ConfVars.HIVE_RPC_QUERY_PLAN, true);
 
     try {
       jc = DagUtils.getInstance().createConfiguration(conf);