samuelcolvin opened a new pull request, #11251: URL: https://github.com/apache/datafusion/pull/11251
## Rationale for this change Until https://github.com/apache/arrow-rs/issues/3821 gets closed and `parse_data_type` gets moved to arrow-rs, it would be extremely helpful for us if `parse_data_type` was public. Currently we have some home rolled logic for inferring arrow datatypes from strings when we define schema in toml files. ## What changes are included in this PR? ```rs datafusion::functions::core::arrow_cast::parse_data_type ``` Is now public ## Are these changes tested? I tried to add a test for this specifically in `datafusion/functions/src/core/arrow_cast.rs` but it works regardless of whether the function is public. Let me know if you'd like me to add a test somewhere else? ## Are there any user-facing changes? ```rs datafusion::functions::core::arrow_cast::parse_data_type ``` Is now public -- 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]
