serkef commented on a change in pull request #5083: [AIRFLOW-4292] Cleanup and 
improve SLA code
URL: https://github.com/apache/airflow/pull/5083#discussion_r284231909
 
 

 ##########
 File path: airflow/jobs.py
 ##########
 @@ -622,6 +622,7 @@ def manage_slas(self, dag, session=None):
             self.log.info("Skipping SLA check for %s because no tasks in DAG 
have SLAs", dag)
             return
 
+        # Find the last executed tasks
 
 Review comment:
   Exactly, I mention on the ticket there are couple of open issues regarding 
this topic and this is my first cleanup attempt, without affecting 
functionality.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to