PHOENIX-3152 Incorrect comparator in QueryOptimizer may cause IllegalArgumentException
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/62d45662 Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/62d45662 Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/62d45662 Branch: refs/heads/4.x-HBase-1.0 Commit: 62d45662be555701a0ca66ea5eb5d920b4b15e04 Parents: 6198bf7 Author: Sergey Soldatov <s...@apache.org> Authored: Sat Aug 6 16:31:13 2016 -0700 Committer: Sergey Soldatov <s...@apache.org> Committed: Sat Aug 6 19:46:23 2016 -0700 ---------------------------------------------------------------------- .../apache/phoenix/optimize/QueryOptimizer.java | 21 +++++++++++--------- 1 file changed, 12 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/phoenix/blob/62d45662/phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java ---------------------------------------------------------------------- diff --git a/phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java b/phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java index e4198ee..bd9c811 100644 --- a/phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java +++ b/phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java @@ -383,35 +383,38 @@ public class QueryOptimizer { // For shared indexes (i.e. indexes on views and local indexes), // a) add back any view constants as these won't be in the index, and // b) ignore the viewIndexId which will be part of the row key columns. - int c = (plan2.getContext().getScanRanges().getBoundPkColumnCount() + (table2.getViewIndexId() == null ? 0 : (boundRanges - 1))) - + int c = (plan2.getContext().getScanRanges().getBoundPkColumnCount() + (table2.getViewIndexId() == null ? 0 : (boundRanges - 1))) - (plan1.getContext().getScanRanges().getBoundPkColumnCount() + (table1.getViewIndexId() == null ? 0 : (boundRanges - 1))); if (c != 0) return c; - if (plan1.getGroupBy()!=null && plan2.getGroupBy()!=null) { + if (plan1.getGroupBy() != null && plan2.getGroupBy() != null) { if (plan1.getGroupBy().isOrderPreserving() != plan2.getGroupBy().isOrderPreserving()) { return plan1.getGroupBy().isOrderPreserving() ? -1 : 1; - } + } } // Use smaller table (table with fewest kv columns) c = (table1.getColumns().size() - table1.getPKColumns().size()) - (table2.getColumns().size() - table2.getPKColumns().size()); if (c != 0) return c; - + // If all things are equal, don't choose local index as it forces scan // on every region (unless there's no start/stop key) - if (table1.getIndexType() == IndexType.LOCAL) { + + if (table1.getIndexType() == IndexType.LOCAL && table2.getIndexType() != + IndexType.LOCAL) { return plan1.getContext().getScanRanges().getRanges().isEmpty() ? -1 : 1; } - if (table2.getIndexType() == IndexType.LOCAL) { + if (table2.getIndexType() == IndexType.LOCAL && table1.getIndexType() != + IndexType.LOCAL) { return plan2.getContext().getScanRanges().getRanges().isEmpty() ? 1 : -1; } // All things being equal, just use the table based on the Hint.USE_DATA_OVER_INDEX_TABLE - if (table1.getType() == PTableType.INDEX) { + + if (table1.getType() == PTableType.INDEX && table2.getType() != PTableType.INDEX) { return comparisonOfDataVersusIndexTable; } - if (table2.getType() == PTableType.INDEX) { + if (table2.getType() == PTableType.INDEX && table1.getType() != PTableType.INDEX) { return -comparisonOfDataVersusIndexTable; } - return 0; }