Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support new OPTIONAL-related operators in cost model #215

Open
hartig opened this issue Aug 2, 2022 · 1 comment
Open

Support new OPTIONAL-related operators in cost model #215

hartig opened this issue Aug 2, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@hartig
Copy link
Member

hartig commented Aug 2, 2022

PRs #213 and #214 have added new logical and new physical operators to add support for OPTIONAL. The implementation of the cost model (see the Java package se.liu.ida.hefquin.engine.queryproc.impl.optimizer.costmodel) does not yet consider these new operators. As a consequence, query plans that contain such an operator cause exceptions during cost estimation. For instance, the class CFRNumberOfTermsShippedInRequests throws an IllegalArgumentException in its initiateCostEstimation function if the root operator of the given plan is such an operator.

The TODO is to extend the implementation of the cost model to also support the new operators.

@hartig hartig added the bug Something isn't working label Aug 2, 2022
@hartig
Copy link
Member Author

hartig commented Aug 6, 2022

Another new operator (PhysicalOpParallelMultiLeftJoin) was added by PR #218.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant