You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by rb...@apache.org on 2019/12/15 22:50:43 UTC

[hive] branch master updated: HIVE-22485: Cross product should set the conf in UnorderedPartitionedKVEdgeConfig (Rajesh Balamohan, reviewed by Ashutosh Chauhan)

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

rbalamohan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new 0ddeb25  HIVE-22485: Cross product should set the conf in UnorderedPartitionedKVEdgeConfig (Rajesh Balamohan, reviewed by Ashutosh Chauhan)
0ddeb25 is described below

commit 0ddeb259c308c710ec795a4dd5b38c2f5c81ce9d
Author: Rajesh Balamohan <rb...@apache.org>
AuthorDate: Mon Dec 16 04:20:25 2019 +0530

    HIVE-22485: Cross product should set the conf in UnorderedPartitionedKVEdgeConfig (Rajesh Balamohan, reviewed by Ashutosh Chauhan)
---
 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java
index 077c94f..7524d7e 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java
@@ -544,7 +544,9 @@ public class DagUtils {
       edgeManagerDescriptor.setUserPayload(cpConfig.toUserPayload(new TezConfiguration(conf)));
       UnorderedPartitionedKVEdgeConfig cpEdgeConf =
         UnorderedPartitionedKVEdgeConfig.newBuilder(keyClass, valClass,
-          ValueHashPartitioner.class.getName()).build();
+          ValueHashPartitioner.class.getName())
+            .setFromConfiguration(conf)
+            .build();
       return cpEdgeConf.createDefaultCustomEdgeProperty(edgeManagerDescriptor);
     case SIMPLE_EDGE:
       // fallthrough