jayzhan211 commented on issue #11513: URL: https://github.com/apache/datafusion/issues/11513#issuecomment-2241157361
> > Is there any type mapping that can't be done without `TypeRelation` 🤔 ? In other words, is there any `arrow::DataType` that > > can be mapped to more than one `LogicalType`? > > I think `DataType::Dictionary(..)` would map to more than one LogicalType (it would map to whatever the dictionary values type was) Why don't we have LogicalType::Dict(LogicalPhysicalFieldRef), similar to List, Struct, and Map. Although `Dict` is not really nested type like those 3, but with `LogicalType::Dict(LogicalPhysicalFieldRef)` we could probably avoid introducing `LogicalPhysicalSchema`? -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
