vandonr-amz commented on issue #32580:
URL: https://github.com/apache/airflow/issues/32580#issuecomment-1634580545
Hmm that's interesting because `execution_timeout` is a property of the base
operator (i.e. of all operators), which is usually handled "automatically", but
I don't think that plays well with deferrable...
Some operators have included it manually in the `defer()` call, with varying
logic, some others don't consider it.
I'm also confused by what's the expected behavior in this report. Setting a
large `execution_timeout` is not going to make the trigger run longer than
`waiter_retries`.
I actually see 3 bugs being reported:
- tasks are successful when triggers time out (probably not what we want ?)
- we hit the trigger timeout even though it should end on max_attempts by
itself 60 seconds before ?
- `execution_timeout` is not handled properly on defer
--
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]
