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

Reply via email to