alamb commented on issue #500:
URL: https://github.com/apache/arrow-rs/issues/500#issuecomment-916449815


   @sum12  this also sounds reasonable to me. 
   
   As you have probably seen the spec for what can be in the timezone string is 
here: https://github.com/apache/arrow/blob/master/format/Schema.fbs#L341-L351 
and I believe your proposal in 
https://github.com/apache/arrow-rs/issues/500#issuecomment-915194224 is 
consistent with that
   
   Regarding `chrono-tz`, please make that an optional dependency (via feature 
flag) of arrow-rs -- we have many users who use arrow in a variety of ways that 
are sensitive to additional dependencies. 
   
   


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


Reply via email to