Skip to content

[C++] Allow timestamp parser to parse offset strings #29023

@asfimport

Description

@asfimport

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:

Note: This issue was originally created as ARROW-13348. Please see the migration documentation for further details.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions