Acfboy commented on issue #20356: URL: https://github.com/apache/datafusion/issues/20356#issuecomment-3935416550
> I would like to work on this by adding early validation for these clauses, following the same error handling pattern used for SELECT expressions via Expr::to_field. Hi @AryanMishra1789, I’m glad to see you’re interested in this! After exploring the codebase further, I’ve noticed that this type checking actually overlaps significantly with what type_coercion already does. Its current placement in the optimization phase seems more like a legacy design choice rather than an oversight, so I’m not entirely sure if adding the check during the logical planning phase is strictly necessary at this point. Maybe you can try to implement your approach and see what kind of feedback or suggestions we get from the rest of the community. -- 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]
