You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ma...@apache.org on 2017/01/27 01:18:53 UTC

[09/26] phoenix git commit: PHOENIX-3516 Performance Issues with queries that have compound filters and specify phoenix.query.force.rowkeyorder=true (addendum)

PHOENIX-3516 Performance Issues with queries that have compound filters and specify phoenix.query.force.rowkeyorder=true (addendum)


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

Branch: refs/heads/calcite
Commit: 07f92732f9c6d2d9464012cebeb4cefc10da95d5
Parents: b326995
Author: Thomas D'Silva <td...@salesforce.com>
Authored: Thu Dec 22 17:29:35 2016 -0800
Committer: Thomas D'Silva <td...@salesforce.com>
Committed: Tue Dec 27 11:44:54 2016 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/phoenix/compile/WhereOptimizer.java  | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/07f92732/phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java b/phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java
index 0681925..5e15f61 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java
@@ -439,7 +439,12 @@ public class WhereOptimizer {
                 if (l.size() == 1) {
                     return l.get(0);
                 }
-                return new AndExpression(l);
+                try {
+                    return AndExpression.create(l);
+                } catch (SQLException e) {
+                    //shouldn't happen
+                    throw new RuntimeException(e);
+                }
             }
             return node;
         }