ferruzzi commented on code in PR #62688:
URL: https://github.com/apache/airflow/pull/62688#discussion_r2956136740


##########
airflow-core/src/airflow/models/deadline.py:
##########
@@ -482,6 +478,36 @@ def deserialize_reference(cls, reference_data: dict):
                 min_runs=min_runs,
             )
 
+    class JobStartDateDeadline(BaseDeadlineReference):
+        """A deadline that returns the start date of the latest active 
SchedulerJob."""
+
+        # By setting this to an empty set, we tell the system this reference
+        # doesn't need an 'id' or any other data from the DAG definition.
+        required_kwargs: set[str] = set()

Review Comment:
   Huh.    The Job table does have a dag_id column, but I think you are right, 
I don't see it actually getting set anywhere.... Without that link I'm not sure 
there is a whole lot of use for this Reference.  We're effectively saying "If 
this Dag hasn't completed within an hour of the last time the scheduler was 
restarted...." and I don't see a particular use case for that.



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