schizophrenicmaniac commented on issue #40860: URL: https://github.com/apache/beam/issues/40860#issuecomment-6461278707
.take-issue Hi, I'd like to work on this. getIcebergTimestampValue in IcebergUtils parses strings with the default ISO formatters, so I'd switch to formatters that also accept a single space in place of T, for both timestamp and timestamptz. I'd add tests in IcebergUtilsTest, including one showing the space-plus-T form is still rejected. -- 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]
