findepi commented on issue #14247: URL: https://github.com/apache/datafusion/issues/14247#issuecomment-2611865233
Using `Field` directly doesn't yet answer the question "how to interpret given field". It's possible to poke at `field.data_type()` ignoring metadata, or check for specific metadata keys that may override how the field should be interpreted. Doing this in every place separately is unwieldy. So we gonna have a helper function that takes a Field and returns... "the type object" representing the type of that field -- either arrow DataType or extension. Given we will have this "the type object" anyway, we should define it upfront and just use in `ReturnTypeArgs`. Maybe we can use the `LogicalType` trait we already have on main for that purpose. -- 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
