paleolimbot commented on PR #18136: URL: https://github.com/apache/datafusion/pull/18136#issuecomment-3716381458
I'm personally happy with this...it seems like there is no opposition to the main change (`DataType` -> `FielRef` in `Cast` and `TryCast`). It might be nice to get all the `Expr` breaking changes out of the way in 52 (I believe Variable was also updated similarly) although I am not in a personal rush. The next steps are to fill in the SQL half (SQL types are already field refs but no built in types have extension types yet) and fill in the extension half (cast registry or something). -- 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]
