alamb commented on PR #10039: URL: https://github.com/apache/datafusion/pull/10039#issuecomment-2073237342
An update here is twofold: 1. There are some very subtle semantics going on 2. As this pass actually changes the types of the plan (on purpose) we need some way to recalculate the schemas (which is what `Plan::with_new_exprs` does, but it also requires a copy of the inputs and exprs, which is not cool I am still putzing with how to make this better -- 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]
