You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/02 17:27:41 UTC

[GitHub] [beam] yirutang commented on pull request #17778: [Beam-14528]: Add ISO time format support for Timestamp, Date, DateTime, Time field.

yirutang commented on PR #17778:
URL: https://github.com/apache/beam/pull/17778#issuecomment-1145118059

   > Please fix unit-test failures - they look related.
   
   PTAL, I double checked backend implementation, and found two things:
   1. it turned the timezone based format support shouldn't be accepted on datetime, time and date. But the datetime does support space.
   2. It seems due to some todo, the space based timezone was based on UTC instead of local time. 
   https://screenshot.googleplex.com/3qnQKNrqsYJgCi8


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org