Stop copying LogicalPlan and Exprs in CommonSubexprEliminate

[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)

Recent profiling (#9637 (comment)) shows that a non trivial amount of plan time happens in CommonSubexprEliminate

Describe the solution you'd like

I would like to reduce the amount of copying in this pass that it no longer appears in profiling traces

Describe alternatives you've considered

No response

Additional context

No response

[alamb]

I believe @peter-toth is working on this one

[peter-toth]

I am, but how this issue relates to #9873? Are they duplicates?

[alamb]

I am, but how this issue relates to #9873? Are they duplicates?

Good call -- I think they are dupes. Closing this one

[peter-toth]

@alamb, why did you reopen this?

[alamb]

@alamb, why did you reopen this?

I am not sure. Sorry about that