dstandish commented on a change in pull request #22184:
URL: https://github.com/apache/airflow/pull/22184#discussion_r825594340



##########
File path: airflow/dag_processing/processor.py
##########
@@ -375,21 +376,16 @@ def manage_slas(self, dag: DAG, session: Session = None) 
-> None:
             .group_by(TI.task_id)

Review comment:
       @ashb @uranusjr  i think we can add a `filter(TI.map_index==-1)` to 
exclude mapped tasks.  this, combined with a check when initializing a mapped 
operator  (check that no SLA  / warn) should effectively disable SLAs for 
mapped tasks i think.... 




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