>From Preetham Poluparthi <[email protected]>:

Attention is currently required from: Ali Alsuliman.

Preetham Poluparthi 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 5:

(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/701f0a8f_235e7ba7?usp=email
 :
PS4, Line 761:                                 
.findFirst().get().isIndexOnlyPlan();
> Is there a chance get() throws NoElement?
Done



--
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: 5
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: Ali Alsuliman <[email protected]>
Gerrit-Comment-Date: Tue, 21 Oct 2025 09:13:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Ali Alsuliman <[email protected]>

Reply via email to