[ARROW-13348] [C++] Allow timestamp parser to parse offset strings

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: C++
• External issue URL: https://github.com/apache/arrow/issues/29023
• Assignee: Unassigned
• Reporter: Weston Pace
• Created: 15/Jul/21 19:29
• Updated: 11/Jan/23 08:32
• Resolved: 23/Nov/21 17:20

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

Issue Links

• is fixed by: ARROW-12820 [C++] Strptime ignores timezone information
• is related to: ARROW-13625 [C++][CSV] Timestamp parsing should accept any valid ISO 8601 without requiring custom parse strings
• is related to: ARROW-12820 [C++] Strptime ignores timezone information