>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/+/20589?usp=email

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

The following approvals got outdated and were removed:
Code-Review+2 by Ali Alsuliman, Contrib+1 by Anon. E. Moose #1000171, 
Verified+1 by Preetham Poluparthi

The change is no longer submittable: Code-Review and Verified are unsatisfied 
now.


Change subject: [ASTERIXDB-3632]: Fix self-join query parsing in index advisor
......................................................................

[ASTERIXDB-3632]: Fix self-join query parsing in index advisor

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

Ext-ref: MB-69430

Details:
Earlier, queries containing self joins were not processed properly by the 
advisor. This update adds proper parsing and handling for self-join queries so 
index recommendations are generated accurately.

Change-Id: Idf20b4c16d729c225798398e5985fc88f07e6bd0
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/indexadvisor/FakeIndexProvider.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/CBOJoinQueries.xml
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.3.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.4.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cbo-join/index-advisor/self-join-advise/self-join-advise.6.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/cbo-join/index-advisor/self-join-advise/self-join-advise.4.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/cbo-join/index-advisor/self-join-advise/self-join-advise.5.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/cbo-join/index-advisor/self-join-advise/self-join-advise.6.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/cbo-join/index-advisor/simple-advise1/simple-advise1.5.adm
14 files changed, 192 insertions(+), 18 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/89/20589/7
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20589?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: Idf20b4c16d729c225798398e5985fc88f07e6bd0
Gerrit-Change-Number: 20589
Gerrit-PatchSet: 7
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]>

Reply via email to