ChickenBenny commented on PR #75387:
URL: https://github.com/apache/airflow/pull/75387#issuecomment-6382747177

   Thanks both! Reworked along Jason's suggestion, so my earlier reply about 
the worker-default fallback is outdated.
   
   - **Python tasks**: the runner sends `task.execution_timeout` to the 
supervisor with `SetExecutionTimeout`, renamed back as Andrushika suggested, so 
both timers use the same value. The fallback and the newsfragment are dropped.
   - **Language SDK tasks**: unchanged.
   - The `default_task_execution_timeout` table is documented in `tasks.rst` 
and `config.yml`.
   - Since #74869 was reverted from `v3-4-test`, its schema changes move to 
`2027-02-28`.
   
   Two things worth a look:
   
   1. Go and TS SDKs built from `main` now need Airflow 3.5, since a 3.4 
supervisor rejects schema `2027-02-28`. Is that OK?
   2. A language SDK task mapped with `expand(execution_timeout=...)` still 
gets `None`. I listed it as a known limitation.


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