You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2018/01/31 05:07:07 UTC
hive git commit: HIVE-17331 : Path must be used as key type of the
pathToAlises (Oleg Danilov via Ashutosh Chauhan)
Repository: hive
Updated Branches:
refs/heads/master 123e2eb64 -> 3e4adaaa3
HIVE-17331 : Path must be used as key type of the pathToAlises (Oleg Danilov via Ashutosh Chauhan)
Signed-off-by: Ashutosh Chauhan <ha...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/3e4adaaa
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/3e4adaaa
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/3e4adaaa
Branch: refs/heads/master
Commit: 3e4adaaa3710a91399b209ab8504214719e220d2
Parents: 123e2eb
Author: Oleg Danilov <ol...@olegd.com>
Authored: Tue Jan 30 21:06:15 2018 -0800
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Tue Jan 30 21:06:15 2018 -0800
----------------------------------------------------------------------
.../org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/3e4adaaa/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java
index 923372d..69447d9 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java
@@ -243,12 +243,12 @@ public final class GenMapRedUtils {
MapredWork plan = (MapredWork) currTask.getWork();
for (int pos = 0; pos < size; pos++) {
String taskTmpDir = taskTmpDirLst.get(pos);
- TableDesc tt_desc = tt_descLst.get(pos);
+ Path taskTmpDirPath = new Path(taskTmpDir);
MapWork mWork = plan.getMapWork();
- if (mWork.getPathToAliases().get(taskTmpDir) == null) {
+ if (!mWork.getPathToAliases().containsKey(taskTmpDirPath)) {
taskTmpDir = taskTmpDir.intern();
- Path taskTmpDirPath = StringInternUtils.internUriStringsInPath(new Path(taskTmpDir));
- mWork.removePathToAlias(taskTmpDirPath);
+ StringInternUtils.internUriStringsInPath(taskTmpDirPath);
+ TableDesc tt_desc = tt_descLst.get(pos);
mWork.addPathToAlias(taskTmpDirPath, taskTmpDir);
mWork.addPathToPartitionInfo(taskTmpDirPath, new PartitionDesc(tt_desc, null));
mWork.getAliasToWork().put(taskTmpDir, topOperators.get(pos));