alamb commented on issue #4680: URL: https://github.com/apache/arrow-datafusion/issues/4680#issuecomment-1359920573
> I'm not entirely sure of the history of this split, but to the uninitiated the split is confusing and frustrating. It also results in a non-trivial amount of schema munging logic to convert to/from the relevant representations I wrote the original `DFSchema` wrapping logic back in the day. If it can be improved that would be great. The key design goal is that DataFusion should *always* use a `DFSchema` (aka a qualified name) so there should not be an implicit conversion from `SchemaRef` --> (implicitly unqualified) `DFSchema` to avoid subtle bugs where the qualifiers are stripped off. Going the other way, `DFSchema` --> `Schema` is 👍 cc @Dandandan @jackwener @mingmwang @andygrove in case you have additional thoughts -- 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]
