We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
EliminateLimit
As part of making the planner faster, we are updating the passes to avoid copying LogicalPlan and Expr (see #9637)
LogicalPlan
Expr
I would like to reduce the amount of copying in this pass (even though it doesn't appear in current profiling)
Apply the model from @Lordworms in #10166 to EliminateLimit.
I think this will be a pretty simple refactor with a good existing pattern, so mark this as good first issue
The text was updated successfully, but these errors were encountered:
Hi, I'd like to take on this issue!
Sorry, something went wrong.
take
kevinmingtarja
Successfully merging a pull request may close this issue.
Is your feature request related to a problem or challenge?
As part of making the planner faster, we are updating the passes to avoid copying
LogicalPlan
andExpr
(see #9637)Describe the solution you'd like
I would like to reduce the amount of copying in this pass (even though it doesn't appear in current profiling)
Describe alternatives you've considered
Apply the model from @Lordworms in #10166 to
EliminateLimit
.Additional context
I think this will be a pretty simple refactor with a good existing pattern, so mark this as good first issue
The text was updated successfully, but these errors were encountered: