paleolimbot opened a new pull request, #18136: URL: https://github.com/apache/datafusion/pull/18136
## Which issue does this PR close? - Closes #18060. I am sorry that I missed the previous PR implementing this ( https://github.com/apache/datafusion/pull/18120 ) and I'm also happy to review that one instead of updating this! ## Rationale for this change Other systems that interact with the logical plan (e.g., SQL, Substrait) can express types that are not strictly within the arrow DataType enum. ## What changes are included in this PR? For the Cast and TryCast structs, the destination data type was changed from a DataType to a FieldRef. ## Are these changes tested? Work in progress! ## Are there any user-facing changes? Yes, any code using `Cast { .. }` to create an expression would need to use `Cast::new()` instead (or pass on field metadata if it has it) -- 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]
