ChickenBenny commented on PR #76103: URL: https://github.com/apache/airflow/pull/76103#issuecomment-6604609350
Hi @shivaam, a correction to my earlier comment. In #76359 I moved the worker-default fallback into the coordinator. When `ti_context.execution_timeout` is unset, the coordinator fills it in with `[core] default_task_execution_timeout` and sends it to the runtime in `StartupDetails`. A `0` stays `0`, meaning no timeout. This way no SDK has to read the config or keep its own fallback, so this PR wouldn't need the `AIRFLOW__CORE__DEFAULT_TASK_EXECUTION_TIMEOUT` change I suggested earlier. The maintainers still need to decide whether this belongs in the supervisor, though, so let's wait for their call before changing anything here. -- 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]
