adriangb opened a new issue, #22399: URL: https://github.com/apache/datafusion/issues/22399
https://github.com/apache/datafusion/pull/22298/ added an optimization for `LogicalPlan` rewriting to avoid cloning the tree when not necessary via a new `map_children_mut` method. We could probably add something similar to `Expr`. I'm less sure about `Arc<dyn PhysicalExpr>` and `Arc<dyn ExecutionPlan>` but we should generally explore our options for this method or other changes we might be able to make to the `TreeNode` API to make optimizer rules more efficient. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
