pierrejeambrun opened a new pull request, #73325: URL: https://github.com/apache/airflow/pull/73325
ADR-0001 (mixed-lang) revised task_id to optional with defaulting to the handler's function name, symmetric with the native-Dag default in ADR-0002. That symmetry is wrong for mixed-lang: Python owns the task_id there, and renaming a TypeScript function must not change which task body Airflow matches. The Go SDK's mixed-lang ADR ([`go-sdk/adr/0007`](https://github.com/apache/airflow/blob/main/go-sdk/adr/0007-mixed-lang-task-handler-interface.md), decision 4) states the same choice for the same reason: > Both dag_id and task_id are written out on TaskHandler definition, because Python owns them; nothing is derived from the Go function name. […] Renaming the Go function must not change which task body Airflow matches. This PR brings the TypeScript stance in line with Go for the mixed-lang case. Native-Dag task-id defaulting (ADR-0002) stays as-is — both ends of every name are TypeScript there. Doc-only. Reverts the mixed-lang portion of #73243. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 4.7) Generated-by: Claude Code (Opus 4.7) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
