Add skills for authoring tasks with Java SDK#234
Conversation
These are for AIP-108, which will be released with Airflow 3.3. We also plan to add skills for Go SDK, so the skills here are structured specifically to avoid duplication across SDKs.
kaxil
left a comment
kaxil
left a comment
There was a problem hiding this comment.
This is in good shape, and notably self-contained: every cross-reference resolves. The four new skills point at each other and at authoring-dags / deploying-airflow / setting-up-astro-project, all of which are already on main. That's the right pattern, and worth calling out because the Go side (#239) references authoring-language-sdk-tasks and the shared config file that only exist in this PR, so those links dangle on main until this merges.
The version hygiene is good too: ${version} placeholders, the -SNAPSHOT/preview note, and the "read the generated Dockerfile, don't pin from memory" hedge all keep it from going stale. For contrast, #239's deploy skill hardcodes apache/airflow:3.3.0 and a @v1.0.0 pin that doesn't resolve, so it could borrow this approach.
The one thing to coordinate is the shared file. This PR and #239 both add skills/configuring-airflow-language-sdks/SKILL.md, and #239's version is a Go-inclusive superset of this Java-framed one. Since this PR is the foundation the Go skills build on, the cleanest path is to land this first and have #239 rebase its copy into a MODIFY that adds the Go coordinator. Noted inline.
Two minor consistency points: the descriptions open imperative (Write / Configure / Build and deploy) where the post-#237 catalog settled on third-person-present (Writes / Configures / Builds and deploys), and the prose uses Dag/Dags while the rest of the catalog uses DAG. Both are consistency nits rather than rules.
Now that the merge order is reversed (Go lands before Java), this PR carries the shared foundation and references no unmerged Java skills: - Duplicate authoring-language-sdk-tasks from #234, framed with the Go SDK as the concrete example (includes the stub pitfalls: empty-body AST check, retry_policy rejection). - Slice configuring-airflow-language-sdks down to the shared wiring plus ExecutableCoordinator; #234 re-adds JavaCoordinator as a MODIFY after rebasing. Port the wording/convention improvements from #234. - Match catalog conventions in the Go skills: third-person descriptions, DAG capitalization in prose.
# Conflicts: # skills/authoring-language-sdk-tasks/SKILL.md # skills/configuring-airflow-language-sdks/SKILL.md
3 participants
These are for AIP-108, which will be released with Airflow 3.3. We also plan to add skills for Go SDK, so the skills here are structured specifically to avoid duplication across SDKs.