You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by gu...@apache.org on 2014/11/12 02:39:43 UTC
svn commit: r1638685 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
Author: gunther
Date: Wed Nov 12 01:39:43 2014
New Revision: 1638685
URL: http://svn.apache.org/r1638685
Log:
HIVE-8811: Dynamic partition pruning can result in NPE during query compilation (Gunther Hagleitner, reviewed by Gopal V)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java?rev=1638685&r1=1638684&r2=1638685&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java Wed Nov 12 01:39:43 2014
@@ -150,6 +150,7 @@ public class TezCompiler extends TaskCom
LOG.info("Found cycle in operator plan...");
cycleFree = false;
removeEventOperator(component);
+ break;
}
}
LOG.info("Cycle free: " + cycleFree);
@@ -227,7 +228,7 @@ public class TezCompiler extends TaskCom
for (Operator<?> child : children) {
if (!indexes.containsKey(child)) {
connect(child, index, nodes, indexes, lowLinks, components);
- lowLinks.put(child, Math.min(lowLinks.get(o), lowLinks.get(child)));
+ lowLinks.put(o, Math.min(lowLinks.get(o), lowLinks.get(child)));
} else if (nodes.contains(child)) {
lowLinks.put(o, Math.min(lowLinks.get(o), indexes.get(child)));
}