malthe commented on pull request #19361: URL: https://github.com/apache/airflow/pull/19361#issuecomment-957748951
@ashb well interestingly, along the lines of the optimization around `DummyOperator` in which the scheduler is able in the common case to immediately mark the task as successful (rather than actually execute the operator which does nothing), to me task-level scheduling (i.e., the proposed `include_date` and `exclude_date` parameters) is also something that belongs as the scheduling level: - It avoids running an operator merely to immediately check the execution date and mark the task as skipped. - The ergonomics are much better. Want to skip Sundays? We could have `exclude_date=SUNDAY` (where `SUNDAY` might simply be a ready-to-use `CronTimeFilter` instance). I think branching operators are best used to make decisions based on the result of a task run. -- 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]
