alamb opened a new pull request, #10356: URL: https://github.com/apache/datafusion/pull/10356
## Which issue does this PR close? Closes https://github.com/apache/datafusion/issues/10210 Part of https://github.com/apache/arrow-datafusion/issues/9637 -- let's make DataFusion planning faster by not copying so much ## Rationale for this change Now that we have the nice TreeNode API thanks to #8913 and @peter-toth let's use it to both simplify the code and avoid copies ## What changes are included in this PR? 1. rewrite `TypeCoercion` via TreeNode API 2. Introduce `LogicalPlan::recompute_schema` to recompute the schema after expressions in a plan are changed ## Are these changes tested? Existing CI ## Are there any user-facing changes? Faster planning TODO benchmark results -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org