>From Ali Alsuliman <[email protected]>:

Attention is currently required from: Preetham Poluparthi.

Ali Alsuliman has posted comments on this change by Preetham Poluparthi. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20475?usp=email )

Change subject: [ASTERIXDB-3635][COMP] Change costing to account for index-only 
plans
......................................................................


Patch Set 4:

(1 comment)

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

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20475/comment/b203eaa4_cafe76d4?usp=email
 :
PS4, Line 761:                                 
.findFirst().get().isIndexOnlyPlan();
Is there a chance get() throws NoElement?



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20475?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: Ifc835f8c4f8386e7b290c68891b4274f1ae87cd5
Gerrit-Change-Number: 20475
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-Attention: Preetham Poluparthi <[email protected]>
Gerrit-Comment-Date: Fri, 17 Oct 2025 17:03:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to