paleolimbot commented on PR #20312: URL: https://github.com/apache/datafusion/pull/20312#issuecomment-3993176784
If anybody would like to review/merge https://github.com/apache/datafusion/pull/20676 (which lets `UUID` SQL get planned into an `Expr`) I'm happy to stack a PR on top of this one implementing the two pieces required to make both of those examples work. If you only need `uuid_field = 'a0eebc99-9c0b-4ef8-bb6d-6bb9bd380a11'`, you can do that today with a custom `ExprPlanner`: https://github.com/apache/datafusion/blob/bcc52cd4c21ea8600101988071a1eccb3e22001c/datafusion/expr/src/planner.rs#L147-L155 The insert is where we could use an injected/registered cast implementation to reduce the change required for the code that currently does this: https://github.com/apache/datafusion/blob/bcc52cd4c21ea8600101988071a1eccb3e22001c/datafusion/sql/src/statement.rs#L2354-L2355 -- 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]
