ahmedabu98 commented on PR #32688: URL: https://github.com/apache/beam/pull/32688#issuecomment-2402624470
Beam (`FieldType.DATETIME`) and Iceberg (`timestamptz`) handle timestamps with timezone in a similar way -- they resolve to UTC and drop the timezone information. I added support for String timestamps to be parsed to OffsetDateTime. But I wonder if it makes sense to allow other types as well (LocalDateTime, joda DateTime, long) and resolve all these types to OffsetDateTime at UTC. This will allow us to have a more unified approach to timestamps - and Iceberg will resolve to UTC anyways -- 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]
