>From Preetham Poluparthi <[email protected]>:
Attention is currently required from: Ali Alsuliman, Preetham Poluparthi.
Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20587?usp=email
to look at the new patch set (#4).
Change subject: [ASTERIXDB-3632]: Fix missing array indexes in current index
detection
......................................................................
[ASTERIXDB-3632]: Fix missing array indexes in current index detection
- user model changes: no
- storage format changes: no
- interface changes: no
Ext-ref: MB-69429
Details:
Previously, in the index advisor, array indexes were not being added to the
list of current indexes even when they already existed, causing them to always
appear under recommended_indexes. This change updates the detection logic so
existing array indexes are correctly recognized and excluded from
recommendations.
Change-Id: I785d6394ef5d328d1edf36fb07e28cd9e143b3b9
---
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/indexadvisor/AdviseIndexRule.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/indexadvisor/FakeIndexProvider.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/array-simple-advise/array-simple-advise.10.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/array-simple-advise/array-simple-advise.11.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/cbo-join/index-advisor/array-simple-advise/array-simple-advise.11.adm
M
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/Index.java
6 files changed, 71 insertions(+), 12 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/87/20587/4
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20587?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I785d6394ef5d328d1edf36fb07e28cd9e143b3b9
Gerrit-Change-Number: 20587
Gerrit-PatchSet: 4
Gerrit-Owner: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Preetham Poluparthi <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>
