[GitHub] [airflow] ecerulm edited a comment on pull request #16631: Change timezone serialization logic

[GitBox <gi...@apache.org>]

ecerulm edited a comment on pull request #16631:
URL: https://github.com/apache/airflow/pull/16631#issuecomment-867680080


   > DAGs do need a timezone, as sometimes often you want a DAG to run at midnight "local" time, not midnight UTC.
   
   If that's the only purpose let me explore the possibility of serializing the timezone as a numeric offset (seconds from utc). Unless there is some interaction with the DST changes. I'll do some tests
   
   EDIT: forget I said that, obviously most timezone are more that just an offset so that's not going to work. I guess if it has a name we need to look it up. And we should check at DAG.__init__ that the timezone is indeed serializable/deserializable to a pendulum.timezone
   


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

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