Dmitry Lychagin has uploaded a new patch set (#2).

Change subject: [ASTERIXDB-2448][COMP] Fix NullPointerException in 
IntroduceSelectAccessMethodRule
......................................................................

[ASTERIXDB-2448][COMP] Fix NullPointerException in 
IntroduceSelectAccessMethodRule

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Fixed NPE when intersecting two indexes and predicate containing
  between operator:
  a) If index intersection plan is not possible then exit the rule
  b) Rewrite between operator into ge/le pair instead of le/le pair

Change-Id: I34d4cde6adf3785feb6a8280fcdb964a5fb72803
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IntroduceSelectAccessMethodRule.java
A 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-secondary-68.sqlpp
A 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index/btree-secondary-68.plan
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/index-selection/intersection-with-between/intersection-with-between.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/index-selection/intersection-with-between/intersection-with-between.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/index-selection/intersection-with-between/intersection-with-between.3.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/index-selection/intersection-with-between/intersection-with-between.4.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/index-selection/intersection-with-between/intersection-with-between.4.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/OperatorExpressionVisitor.java
10 files changed, 233 insertions(+), 15 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/37/2937/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2937
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I34d4cde6adf3785feb6a8280fcdb964a5fb72803
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: stabilization-f69489
Gerrit-Owner: Dmitry Lychagin <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>

Reply via email to