>From Glenn Galvizo <[email protected]>:

Attention is currently required from: Glenn Galvizo.
Hello Jenkins, Anon. E. Moose #1000171,

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17438

to look at the new patch set (#2).

Change subject: [ASTERIXDB-3145][COMP] Fix for subplan-JOIN rewrite type bug
......................................................................

[ASTERIXDB-3145][COMP] Fix for subplan-JOIN rewrite type bug

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

Details:
A small change: issue was that JoinFromSubplanRewrite was incorrectly
modifying the original SELECT predicate (used for the JOIN), meaning
that we would leave the plan in an inconsistent state if we didn't
apply INLJ (for array indexes).

Change-Id: I134ac050dad5a9d9b1956caba1457778cb77f33e
---
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/array-index/atomic-and-array-queries/query8.plan
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/array/JoinFromSubplanRewrite.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/array-index/atomic-and-array-queries/query8.plan
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/array/AbstractOperatorFromSubplanRewrite.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/array-index/atomic-and-array-queries/query9.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/array-index/atomic-and-array-queries/query9.plan
6 files changed, 49 insertions(+), 26 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/38/17438/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17438
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: neo
Gerrit-Change-Id: I134ac050dad5a9d9b1956caba1457778cb77f33e
Gerrit-Change-Number: 17438
Gerrit-PatchSet: 2
Gerrit-Owner: Glenn Galvizo <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Glenn Galvizo <[email protected]>
Gerrit-MessageType: newpatchset

Reply via email to