jorisvandenbossche commented on issue #39315: URL: https://github.com/apache/arrow/issues/39315#issuecomment-2109988129
The main issue (as was discussed in the original issue https://github.com/apache/arrow/issues/30117, before we closed that after adding the "Z" suffix) is that showing the local timezone requires a timezone database to be present, and this is not guaranteed. (I think the requirement on Compute could probably be fixed, by moving or replicating the logic of `local_timestamp` to our pretty printing utilities) At the time the original issue was discussed, the tz database wasn't yet supported for Windows, but that has improved nowadays (although the user still need to download it manually and put it in the correct location or point pyarrow to it). We could decide to actually print wall time _if_ a tzdb is available, and otherwise still fall back on showing the UTC values with "Z" suffix. That would be an annoying inconsistency in the pretty printing, but at least make it less confusing for many users on linux/mac. -- 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]
