Skip to content

Stop copying LogicalPlan and Exprs in EliminateLimit #10212

Closed
@alamb

Description

@alamb

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 and Expr (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

Metadata

Metadata

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions