>From Peeyush Gupta <[email protected]>: Attention is currently required from: Ali Alsuliman, Preetham Poluparthi, Shahrzad Shirazi.
Peeyush Gupta has posted comments on this change by Shahrzad Shirazi. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20774?usp=email ) Change subject: [ASTERIXDB-3688][COMP] Improving Index Selection for Index-Only Query Plans ...................................................................... Patch Set 7: (1 comment) File asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20774/comment/98d5e16a_9674202e?usp=email : PS7, Line 307: else if (!appliedIndexTypes.contains(indexType)) { : appliedIndexTypes.add(indexType); : result.add(new IntroduceSelectAccessMethodRule.IndexAccessInfo(chosenAccessMethod, : chosenIndex)); : } If a covering index is found and then we iterate over a non covering index then the else if condition will still add it to the result list. Is this logic ok? or in case we have found a covering index we don't want to consider any non-covering index. -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20774?usp=email To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I4f996ee7b6eb03067fb384161fb4eb60f94e8353 Gerrit-Change-Number: 20774 Gerrit-PatchSet: 7 Gerrit-Owner: Shahrzad Shirazi <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Peeyush Gupta <[email protected]> Gerrit-Reviewer: Preetham Poluparthi <[email protected]> Gerrit-Attention: Shahrzad Shirazi <[email protected]> Gerrit-Attention: Ali Alsuliman <[email protected]> Gerrit-Attention: Preetham Poluparthi <[email protected]> Gerrit-Comment-Date: Thu, 15 Jan 2026 04:22:45 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
