paleolimbot commented on PR #20312: URL: https://github.com/apache/datafusion/pull/20312#issuecomment-4095658895
I gave it a go in https://github.com/apache/datafusion/pull/21071, which gets a cast to and from UUID in SQL. All details up for grabs in that PR...it was mostly a plumbing exercise to find all the fun places that we need to pipe through the registry. Maybe https://github.com/apache/datafusion/pull/21071/changes/e275152093bfa1f84c657152308f05bab63d50e1..df2db626622f4b6ed2424ffce318cb5b7e435869 will give a slightly more relevant set of changes. > And then the planner could consider i) the defaults from the extension types and ii) any custom registered casting functions. It's a good point that another way to do this is to have a dedicated cast registry that considers both types and dispatches based on that. Overriding other kinds of casts is a more niche use case but can be done with an optimizer rule instead of yet another registry to pipe through. -- 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]
