ashb commented on code in PR #46584: URL: https://github.com/apache/airflow/pull/46584#discussion_r1952401465
########## airflow/models/skipmixin.py: ########## @@ -120,23 +74,11 @@ def skip( return task_ids_list = [d.task_id for d in task_list] - SkipMixin._set_state_to_skipped(dag_id, run_id, task_ids_list, session) Review Comment: And yeah, I think we should do that. The TIDep skipping again is I think do catch it when things get cleared but the previous skip result is maintained -- see #7276, but we should still skip tasks immediately in the API server handling path. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org