You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by pr...@apache.org on 2014/10/24 01:06:13 UTC

svn commit: r1633979 - /hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java

Author: prasanthj
Date: Thu Oct 23 23:06:12 2014
New Revision: 1633979

URL: http://svn.apache.org/r1633979
Log:
HIVE-8576: Guaranteed NPE in StatsRulesProcFactory (Lars Francke via Prasanth J)

Modified:
    hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java

Modified: hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java?rev=1633979&r1=1633978&r2=1633979&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java (original)
+++ hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java Thu Oct 23 23:06:12 2014
@@ -1394,7 +1394,7 @@ public class StatsRulesProcFactory {
      */
     private List<Integer> getPrimaryKeyCandidates(List<Operator<? extends OperatorDesc>> ops) {
       List<Integer> result = Lists.newArrayList();
-      if (ops != null || !ops.isEmpty()) {
+      if (ops != null && !ops.isEmpty()) {
         for (int i = 0; i < ops.size(); i++) {
           Operator<? extends OperatorDesc> op = ops.get(i);
           if (op instanceof ReduceSinkOperator) {