jscheffl commented on code in PR #53492:
URL: https://github.com/apache/airflow/pull/53492#discussion_r2250073077
##########
airflow-core/src/airflow/models/taskinstance.py:
##########
@@ -554,6 +559,14 @@ class TaskInstance(Base, LoggingMixin):
innerjoin=True,
viewonly=True,
)
+ pool_model: Pool = relationship(
+ "Pool",
+ primaryjoin="TaskInstance.pool == Pool.pool",
+ foreign_keys=pool,
+ uselist=False,
+ innerjoin=True,
Review Comment:
At the moment there is no constraint on the relation between pool and TI.
Would it mean the scheduler will not "see" the TIs when a pool is referenced
that an admin has deleted? Do we need to add a constraint to pool table as well?
--
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]
