-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-XXX] Clarify daylight savings time behaviour #6324
Conversation
Hey @jmcarp ! Welcome back! I think there are some backtick problems in .rst files :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing the backticks :)
docs/timezone.rst
Outdated
regardless if day light savings time is in place. | ||
|
||
Time zone aware DAGs that use cron schedules respect daylight savings | ||
time. For example, a DAG with a start date in the `US/Eastern` time zone |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
time. For example, a DAG with a start date in the `US/Eastern` time zone | |
time. For example, a DAG with a start date in the ``US/Eastern`` time zone |
docs/timezone.rst
Outdated
|
||
Time zone aware DAGs that use cron schedules respect daylight savings | ||
time. For example, a DAG with a start date in the `US/Eastern` time zone | ||
with a schedule of `0 0 * * *` will run daily at 04:00 UTC during |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with a schedule of `0 0 * * *` will run daily at 04:00 UTC during | |
with a schedule of ``0 0 * * *`` will run daily at 04:00 UTC during |
docs/timezone.rst
Outdated
specify a ``timedelta(hours=2)`` you will always want to run two hours later. In this case day light savings time will | ||
be taken into account. | ||
|
||
Time zone aware DAGs that use `timedelta` or `relativedelta` schedules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Time zone aware DAGs that use `timedelta` or `relativedelta` schedules | |
Time zone aware DAGs that use ``timedelta`` or ``relativedelta`` schedules |
docs/timezone.rst
Outdated
Time zone aware DAGs that use `timedelta` or `relativedelta` schedules | ||
respect daylight savings time for the start date but do not adjust for | ||
daylight savings time when scheduling subsequent runs. For example, a | ||
DAG with a start date of `pendulum.create(2020, 1, 1, tz="US/Eastern")` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DAG with a start date of `pendulum.create(2020, 1, 1, tz="US/Eastern")` | |
DAG with a start date of ``pendulum.create(2020, 1, 1, tz="US/Eastern")`` |
docs/timezone.rst
Outdated
respect daylight savings time for the start date but do not adjust for | ||
daylight savings time when scheduling subsequent runs. For example, a | ||
DAG with a start date of `pendulum.create(2020, 1, 1, tz="US/Eastern")` | ||
and a schedule interval of `timedelta(days=1)` will run daily at 05:00 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and a schedule interval of `timedelta(days=1)` will run daily at 05:00 | |
and a schedule interval of ``timedelta(days=1)`` will run daily at 05:00 |
c184e6b
to
0ada503
Compare
(cherry picked from commit 52c2be5)
(cherry picked from commit 52c2be5)
(cherry picked from commit 52c2be5)
(cherry picked from commit 52c2be5)
Make sure you have checked all steps below.
Jira
Description
Tests
Commits
Documentation