>From Ritik Raj <[email protected]>:

Attention is currently required from: Ali Alsuliman.

Ritik Raj has posted comments on this change by Ritik Raj. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20410?usp=email )

Change subject: [ASTERIXDB-3650][COMP] Ignore sample indexes when checking for 
secondary indexes
......................................................................


Patch Set 4: Code-Review+1

(2 comments)

File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/PushFieldAccessRule.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20410/comment/d4cacc07_f682018b?usp=email
 :
PS3, Line 155:         List<Index> datasetIndexes =
> Add the following to MetadataProvider: […]
Done


File 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20410/comment/61635d0e_acc4ee58?usp=email
 :
PS3, Line 488:         boolean hasSecondaries = 
hasNonSampleSecondaryIndex(dataset, metadataProvider);
> Change to: […]
Done



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20410?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: phoenix
Gerrit-Change-Id: I65ca9c74192d14bbaecf8a306902e4cecf1337d2
Gerrit-Change-Number: 20410
Gerrit-PatchSet: 4
Gerrit-Owner: Ritik Raj <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Ritik Raj <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Comment-Date: Fri, 17 Oct 2025 05:31:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Ali Alsuliman <[email protected]>

Reply via email to