hussein-awala commented on code in PR #32990:
URL: https://github.com/apache/airflow/pull/32990#discussion_r1304779062
##########
airflow/models/taskinstance.py:
##########
@@ -426,6 +436,10 @@ class TaskInstance(Base, LoggingMixin):
# migration, we are keeping it as DateTime pending a change where expensive
# migration is inevitable.
+ # The reason why the trigger should time out after trigger_timeout
+ # (e.g. "execution_timeout", "sensor_timeout", "trigger_timeout")
+ trigger_timeout_reason = Column(String(256))
Review Comment:
> does seem to make more sense on trigger. but not sure if we should break
with what's there already. just, ti tends to be one of the larger tables, and
it's already pretty wide.
it depends on if we decide to handle the timeout in the task instance as I
do now, or the trigger cleanup as you propose.
--
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]
