-
Notifications
You must be signed in to change notification settings - Fork 17.1k
Cross-SDK DagSerialization compatibility CI for the Go SDK #66942
Copy link
Copy link
Open
Labels
area:CIAirflow's tests and continious integrationAirflow's tests and continious integrationarea:dev-toolsgo-sdkLabel to track work items for golang task sdkLabel to track work items for golang task sdkkind:featureFeature RequestsFeature Requestskind:taskA task that needs to be completed as part of a larger issueA task that needs to be completed as part of a larger issuepriority:mediumBug that should be fixed before next release but would not block a releaseBug that should be fixed before next release but would not block a release
Metadata
Metadata
Assignees
Labels
area:CIAirflow's tests and continious integrationAirflow's tests and continious integrationarea:dev-toolsgo-sdkLabel to track work items for golang task sdkLabel to track work items for golang task sdkkind:featureFeature RequestsFeature Requestskind:taskA task that needs to be completed as part of a larger issueA task that needs to be completed as part of a larger issuepriority:mediumBug that should be fixed before next release but would not block a releaseBug that should be fixed before next release but would not block a release
Type
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
No status
Background
ADR 0003 explicitly promises:
The Go
serdepackage exists(
go-sdk/pkg/execution/serde.go,serde_test.go) but the cross-SDKparity hook does not - there is no prek/CI step that runs the
Go serializer against the shared
test_dags.yamland diffs the outputagainst Python's
SerializedDAG.serialize_daglike the Java SDK doesvia
compare.py.What needs to happen
validation/serialization/test_dags.yamland emits a JSON fileper Dag fixture (parallel to the Java
SerializationCompatibilityTest).scripts/ci/prek/so the existingcompare.pyrunsagainst
serialized_go.jsonalongsideserialized_java.json.Acceptance criteria
SerializedDAG.serialize_dagshape that Go doesn'tfollow fails CI before merge.
Context
go-sdk/adr/0003-coordinator-protocol-msgpack-ipc.md.check_java_serialization_compatibility.py; we extend the samepattern.
Drafted-by: Claude Code (Opus 4.7) (no human review before posting)