tustvold commented on PR #3101: URL: https://github.com/apache/arrow-rs/pull/3101#issuecomment-1314868053
> 4. consider the timezone I originally was going to suggest this, but I decided against it as the semantics are actually a little bit funky. In particular, courtesy of the wonders of daylight savings, a non-FixedOffset timezone requires the date in order to be interpreted. I think it is acceptable to only handle timezones for timestamps, and not for times. FWIW this is the same approach taken by chrono - there is `DateTime<Tz>` and `NaiveDateTime` but only `NaiveTime` and no `Time<Tz>` -- 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]
