phoenix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jamestay...@apache.org
Subject [3/3] phoenix git commit: PHOENIX-2888 Aggregations for similar IN list expressions doesn't work correctly (Sergey Soldatov)
Date Fri, 13 May 2016 00:21:43 GMT
PHOENIX-2888 Aggregations for similar IN list expressions doesn't work correctly (Sergey Soldatov)


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/b9a85747
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/b9a85747
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/b9a85747

Branch: refs/heads/4.x-HBase-0.98
Commit: b9a857472b6e99203cf3424b76c921f218af7456
Parents: bddc75f
Author: James Taylor <jamestaylor@apache.org>
Authored: Thu May 12 14:34:47 2016 -0700
Committer: James Taylor <jamestaylor@apache.org>
Committed: Thu May 12 17:21:42 2016 -0700

----------------------------------------------------------------------
 .../java/org/apache/phoenix/expression/InListExpression.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/b9a85747/phoenix-core/src/main/java/org/apache/phoenix/expression/InListExpression.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/expression/InListExpression.java
b/phoenix-core/src/main/java/org/apache/phoenix/expression/InListExpression.java
index a4a9353..a977f1f 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/expression/InListExpression.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/expression/InListExpression.java
@@ -175,7 +175,7 @@ public class InListExpression extends BaseSingleExpression {
     public int hashCode() {
         final int prime = 31;
         int result = 1;
-        result = prime * result + values.hashCode();
+        result = prime * result + children.hashCode() + values.hashCode();
         return result;
     }
 
@@ -185,7 +185,7 @@ public class InListExpression extends BaseSingleExpression {
         if (obj == null) return false;
         if (getClass() != obj.getClass()) return false;
         InListExpression other = (InListExpression)obj;
-        if (!values.equals(other.values)) return false;
+        if (!children.equals(other.children) || !values.equals(other.values)) return false;
         return true;
     }
 


Mime
View raw message