eladkal commented on code in PR #41260:
URL: https://github.com/apache/airflow/pull/41260#discussion_r1703934310


##########
airflow/models/taskinstance.py:
##########
@@ -3892,9 +3892,12 @@ def schedule_downstream_tasks(self, session: Session = 
NEW_SESSION, max_tis_per_
 
         :meta: private
         """
-        return TaskInstance._schedule_downstream_tasks(
-            ti=self, session=session, max_tis_per_query=max_tis_per_query
-        )
+        try:
+            return TaskInstance._schedule_downstream_tasks(
+                ti=self, session=session, max_tis_per_query=max_tis_per_query
+            )
+        except Exception:
+            self.log.exception("Error scheduling downstream tasks. Skipping 
it")

Review Comment:
   Does this call for action from user side?
   The thing about logging such exceptions is that users want to solve the 
underlying issue but from the description it sounds like there is no real 
action item to avoid it? Should we leave hints in the message (for example 
consider disable mini scheduler)



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

Reply via email to