-
Notifications
You must be signed in to change notification settings - Fork 4k
Description
Neither the ISO8601 parser nor the strptime parser support parsing timezone offset strings (e.g. 2017-08-19 12:22:11.802755+00). These are sometimes emitted by other tools (e.g. Postgresql) and should be supported for compatibility. I think it's acceptable for them to parse to Timestamp(units, "UTC") or Timestamp(units, offset) whichever is more convenient.
This issue doesn't necessarily require supporting tzdata timezone names (e.g. "America/Denver", etc.) and those technically aren't ISO8601 compliant anyways.
Support should be added to both the ISO-8601 parser (which should support 00:00, 0000, and 00) and the strptime parser (which should add support for %z).
Reporter: Weston Pace / @westonpace
Watchers: Rok Mihevc / @rok
Related issues:
- [C++] Strptime ignores timezone information (is fixed by)
- [C++][CSV] Timestamp parsing should accept any valid ISO 8601 without requiring custom parse strings (is related to)
- [C++] Strptime ignores timezone information (is related to)
Note: This issue was originally created as ARROW-13348. Please see the migration documentation for further details.