Acfboy commented on PR #20086: URL: https://github.com/apache/datafusion/pull/20086#issuecomment-3827344104
Hi @Jefffrey, thanks for the feedback! My apologies, I didn't consider this thoroughly enough. I realize now that making this change directly (during the initial conversion from statement to LogicalPlan) would break the optimizer. For instance, I haven't even migrated the logic from simplify_expr to the UDF's simplify method yet. I will convert this pr to a draft for now and carefully study the impact of this change on existing optimization rules. -- 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]
