[GitHub] [arrow-rs] tustvold commented on issue #4472: Add an ExtensionType to DataType enum

["tustvold (via GitHub)" <gi...@apache.org>]

tustvold commented on issue #4472:
URL: https://github.com/apache/arrow-rs/issues/4472#issuecomment-1615194168

   > but when exporting/importing an array over the C data interface, the extension type metadata is lost
   
   I might be missing something here, but why would it be lost, schema metadata should roundtrip over C data interface?
   
   > I'd propose adding a new enum variant to DataType:
   
   My major objection to this approach is that it undermines the transparency of extension types. In particular array downcasting logic must now be updated to "see through" DataType::Extension, and arrays must be updated to be able to propagate this information correctly. I would much prefer an approach that does not require this. 


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org