fordN commented on PR #23459: URL: https://github.com/apache/datafusion/pull/23459#issuecomment-4938483064
@xudong963 here's a follow up to https://github.com/apache/datafusion/pull/23395 that expands the set of expressions to not be "re-inline'd" to include performance considerations along with the correctness considerations. In practice, this change has shown to have a significant effect on the performance of queries where outer statement references expensive computation results from inner. -- 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]
