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 2014/08/11 05:06:41 UTC

svn commit: r1617197 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java

Author: hashutosh
Date: Mon Aug 11 03:06:41 2014
New Revision: 1617197

URL: http://svn.apache.org/r1617197
Log:
HIVE-7521 : Reference equality is used on Boolean in NullScanOptimizer#WhereFalseProcessor#process() (Kang HS via Ashutosh Chauhan)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java?rev=1617197&r1=1617196&r2=1617197&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanOptimizer.java Mon Aug 11 03:06:41 2014
@@ -86,7 +86,7 @@ public class NullScanOptimizer implement
         return null;
       }
       ExprNodeConstantDesc c = (ExprNodeConstantDesc) condition;
-      if (c.getValue() != Boolean.FALSE) {
+      if (!Boolean.FALSE.equals(c.getValue())) {
         return null;
       }