[GitHub] [arrow-julia] ericphanson commented on pull request #482: move timezone support to extension

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

ericphanson commented on PR #482:
URL: https://github.com/apache/arrow-julia/pull/482#issuecomment-1680365383

   > This is almost assuredly a breaking change because deserialization behavior will change when TimeZones isn't loaded either directly or through some other package
   
   Regardless of breakingness, I'm not sure this is a good idea, since the [Arrow format itself has a notion of `timezone`](https://github.com/apache/arrow/blob/641201416c1075edfd05d78b539275065daac31d/format/Schema.fbs#L360), I think it makes sense that Arrow.jl itself can deserialize a table with such types (e.g. coming from another language), without needing to load another package.
   
   IMO it would be better to try to fix invalidation / other sources of re-compilation in TimeZones itself, or otherwise improve it's load time, rather than making it optional.


-- 
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