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:42:15 UTC
svn commit: r1638686 -
/hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
Author: gunther
Date: Wed Nov 12 01:42:15 2014
New Revision: 1638686
URL: http://svn.apache.org/r1638686
Log:
HIVE-8811: Dynamic partition pruning can result in NPE during query compilation (Gunther Hagleitner, reviewed by Gopal V)
Modified:
hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
Modified: hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java?rev=1638686&r1=1638685&r2=1638686&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java (original)
+++ hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java Wed Nov 12 01:42:15 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)));
}