>From Shahrzad Haji Amin Shirazi <[email protected]>:

Attention is currently required from: Shahrzad Haji Amin Shirazi.
Hello Ali Alsuliman, Jenkins, Anon. E. Moose #1000171,

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

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

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

Change subject: [ASTERIXDB-XXXX][COMP] Enable index only plan when index 
includes UNKNOWNs
......................................................................

[ASTERIXDB-XXXX][COMP] Enable index only plan when index includes UNKNOWNs


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

Details:
When a secondary index includes UNKNOWNs, the original SELECT condition needs
to be applied after the secondary index search to ensure NULLs/MISSINGs are
filtered out. For example, the predicate in the below query can incorrectly
return NULLs/MISSINGs matches:
SELECT id, s_f2 FROM ds7 WHERE s_f2 < "4" ORDER BY id;

Change-Id: I457606d2b0b001432a7607407d6755130e49acc1
---
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index-join/leftouterjoin-probe-pidx-with-join-btree-sidx_03-index-only.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index-selection/cast-default-null/cast-default-null-24.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index-join/leftouterjoin-probe-pidx-with-join-btree-sidx_01_ps.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-sidx-idxonly-10.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index-join/leftouterjoin-probe-pidx-with-join-btree-sidx_03-index-only.plan
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/index-selection/cast-default-null/cast-default-null.23.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-sidx-idxonly-01_ps.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index-selection/btree-index-composite-key-04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index/btree-sidx-idxonly-01.plan
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/metrics/secondary-index-index-only/secondary-index-index-only.1.ddl.sqlpp
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index/btree-sidx-idxonly-01_ps.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index-join/leftouterjoin-probe-pidx-with-join-btree-sidx_01_ps.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/btree-index-selection/cast-default-null/cast-default-null-24.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index/btree-sidx-idxonly-01.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/btree-index-selection/btree-index-composite-key-04.sqlpp
16 files changed, 134 insertions(+), 91 deletions(-)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I457606d2b0b001432a7607407d6755130e49acc1
Gerrit-Change-Number: 17629
Gerrit-PatchSet: 4
Gerrit-Owner: Shahrzad Haji Amin Shirazi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Shahrzad Haji Amin Shirazi <[email protected]>
Gerrit-MessageType: newpatchset

Reply via email to