Skip to content

Commit

Permalink
[AIRFLOW-6860] Default ignore_first_depends_on_past to True (#7610)
Browse files Browse the repository at this point in the history
  • Loading branch information
zhongjiajie authored Mar 20, 2020
1 parent 5106a29 commit 9575706
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion UPDATING.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ with DAG('my_dag):

### Deprecating ignore_first_depends_on_past on backfill command and default it to True

When doing backfill with `depends_on_past` dags, users will need to pass `ignore_first_depends_on_past`.
When doing backfill with `depends_on_past` dags, users will need to pass `--ignore-first-depends-on-past`.
We should default it as `true` to avoid confusion

### Custom executors is loaded using full import path
Expand Down
2 changes: 1 addition & 1 deletion airflow/cli/commands/dag_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ def dag_backfill(args, dag=None):
signal.signal(signal.SIGTERM, sigint_handler)

import warnings
warnings.warn('--ignore_first_depends_on_past is deprecated as the value is always set to True',
warnings.warn('--ignore-first-depends-on-past is deprecated as the value is always set to True',
category=PendingDeprecationWarning)

if args.ignore_first_depends_on_past is False:
Expand Down

0 comments on commit 9575706

Please sign in to comment.