o-nikolas commented on code in PR #30467:
URL: https://github.com/apache/airflow/pull/30467#discussion_r1157845180


##########
docs/apache-airflow/authoring-and-scheduling/timezone.rst:
##########
@@ -150,6 +150,13 @@ date will be converted to UTC using the timezone 
associated with ``start_date``
 or ``end_date``, then for calculations this timezone information will be
 disregarded.
 
+.. note::
+    When authoring Timezone aware DAG you must make sure that the underlying 
timezone library (for example: pendulum)
+    is updated with changes to regulations (daylight saving changes etc...). 
When moving time
+    is expected you should verify with the underlying timezone library that 
switch will happen as expected.
+    There might be a need to update the library version. As a general 
recommendation if you can author DAGs in UTC
+    that is preferred.

Review Comment:
   ```suggestion
       When authoring a Timezone aware DAG you must make sure that the 
underlying timezone library (for example: pendulum)
       is updated with recent changes to regulations (daylight saving changes 
etc...). When a change in time
       is expected you should verify with the underlying timezone library that 
the switch will happen as expected.
       There might be a need to update the library version. As a general 
recommendation if you can author DAGs in UTC
       that is preferred.
   ```



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

Reply via email to