Skip to content

Correct the example config for the coordinators - #68929

Merged
jason810496 merged 2 commits into
apache:mainfrom
jason810496:fix/coordinator/config-template-jars-root
Jun 24, 2026
Merged

Correct the example config for the coordinators#68929
jason810496 merged 2 commits into
apache:mainfrom
jason810496:fix/coordinator/config-template-jars-root

Conversation

@jason810496

@jason810496jason810496 commented Jun 24, 2026

Copy link
Copy Markdown
Member

What

  • We didn't mention the required jars_root kwargs in the config template.
  • Fix the outdated docstring as well (we used [{"name": "go", "classpath": "..."}] before, but we promote the coordinator identifier from list to dict as "go": {"classpath": "..."}, "java": {"claspath": "..."}).

@jason810496jason810496 self-assigned this Jun 24, 2026
@jason810496jason810496 added this to the Airflow 3.3.0 milestone Jun 24, 2026
phanikumv
phanikumv reviewed Jun 24, 2026
View reviewed changes
Comment threadtask-sdk/src/airflow/sdk/coordinators/java/coordinator.py
@jason810496
jason810496 merged commit 4aafb95 into apache:mainJun 24, 2026
102 checks passed
jason810496 added a commit that referenced this pull request Jun 25, 2026
@jason810496
[v3-3-test] Correct the example config for the coordinators (#68929) (#…
e9f61f4
…68940)
* Correct the example and docstring for the coordinators
* Guard the config.yml coordinators example against drift
(cherry picked from commit 4aafb95)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phanikumvphanikumvphanikumv approved these changes
@ashbashbAwaiting requested review from ashb
@kaxilkaxilAwaiting requested review from kaxil
@amoghrajeshamoghrajeshAwaiting requested review from amoghrajesh
@uranusjruranusjrAwaiting requested review from uranusjruranusjr is a code owner

Assignees

@jason810496jason810496

Labels

area:ConfigTemplatesarea:task-sdk

Projects

None yet

Milestone

Airflow 3.3.0

Development

Successfully merging this pull request may close these issues.

2 participants

@jason810496@phanikumv