>From Ali Alsuliman <[email protected]>:

Attention is currently required from: Peeyush Gupta, Vijay Sarathy, 
[email protected].
Ali Alsuliman has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19383 )

Change subject: [ASTERIXDB-3555][COMP] Use Join Samples to get Join Selectivity
......................................................................


Patch Set 10:

(1 comment)

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

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19383/comment/e934e25b_36a2506c
PS5, Line 262: joinEnum.allJoinOps.get(0);
> Yes, you are right. I need to pick an inner join only. […]
Isn't it possible to store the corresponding join op along side the join 
condition in findJoinConditionsAndAssignSels()?
There you can store the JoinOperator inside the JoinCondition. Then, when you 
call stats.getSelectivityFromAnnotationMain(), you pass along the JoinOperator.



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19383
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I863dc378d5ead9f8b48f09f7bc400f5bdaba4839
Gerrit-Change-Number: 19383
Gerrit-PatchSet: 10
Gerrit-Owner: [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: Vijay Sarathy <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Attention: Vijay Sarathy <[email protected]>
Gerrit-Attention: [email protected]
Gerrit-Comment-Date: Tue, 11 Feb 2025 03:52:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Ali Alsuliman <[email protected]>
Comment-In-Reply-To: [email protected]
Gerrit-MessageType: comment

Reply via email to