uranusjr commented on a change in pull request #16475:
URL: https://github.com/apache/airflow/pull/16475#discussion_r723801595



##########
File path: airflow/cli/cli_parser.py
##########
@@ -909,7 +917,9 @@ class GroupCommand(NamedTuple):
             "Run subsections of a DAG for a specified date range. If 
reset_dag_run option is used, "
             "backfill will first prompt users whether airflow should clear all 
the previous dag_run and "
             "task_instances within the backfill date range. If 
rerun_failed_tasks is used, backfill "
-            "will auto re-run the previous failed task instances  within the 
backfill date range"
+            "will auto re-run the previous failed task instances  within the 
backfill date range. If "

Review comment:
       ```suggestion
               "will auto re-run the previous failed task instances within the 
backfill date range. If "
   ```

##########
File path: airflow/cli/commands/dag_command.py
##########
@@ -122,13 +123,15 @@ def dag_backfill(args, dag=None):
                 verbose=args.verbose,
                 conf=run_conf,
                 rerun_failed_tasks=args.rerun_failed_tasks,
+                rerun_succeeded_task=args.rerun_succeeded_tasks,
                 run_backwards=args.run_backwards,
             )
         except ValueError as vr:
             print(str(vr))
             sys.exit(1)
 
 
+

Review comment:
       I think linter would complain about this extra empty line?




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