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 https://github.com/apache/datafusion/issues/9637)

Recent profiling (https://github.com/apache/datafusion/issues/9637#issuecomment-2073325456) 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 https://github.com/apache/datafusion/issues/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