The GitHub Actions job "Tests (AMD)" on airflow.git/main has succeeded. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: a2a78e6839fafe1aedf8969ec2606aca54699757 / Jason(Zhe-You) Liu <[email protected]> Generate data models from supervisor schema for Go-SDK (#68629) * Generate data models from supervisor schema for Go-SDK * Add EnsureType to bind the Type field at encodeRequest level * Fix the default incomming map_index * Harden type-discriminator binding and error decoding for Go-SDK coordinator protocol Outbound messages rely on a single chokepoint (EnsureType) to stamp the wire "type" discriminator. A pointer body slipped through that switch unstamped, producing a frame the supervisor could not dispatch; it now stamps the pointee so the binding holds regardless of how a caller passes the body. Supervisor error replies also lost their typed error code when the detail payload was off-contract, degrading specific errors (e.g. variable-not-found) into a generic one; the code is now recovered independently so callers still see the right typed error. * Respect the default values at auto generated stage * Respect explicit nil case * Drop dead discriminator-tracking state from genmodels generator The HasType flag and its wrapper struct recorded which parsed structs declared a Type discriminator field, but discriminator emission now keys off the schema's "type" const rather than the parsed struct, so the flag was set and never read. Collapsing the per-struct metadata to a plain field slice removes the unread state with no change to generated output. * Add supervisor schema bump guide * Preserve explicit zero on outbound integer defaults in Go-SDK models A plain int field with ,omitempty drops an explicit 0 on encode, so the supervisor reapplies its non-zero schema default (e.g. treating a real map_index 0 as the -1 unmapped sentinel). The generator now widens any concrete integer field carrying a non-zero schema default to a pointer, so ,omitempty keeps meaning "absent" for nil while an explicit zero still encodes; DecodeMsgpack default-seeding skips these fields since nil is already the correct "use the default" value on decode. * Address self-review nits * Widen types for float and book as well Report URL: https://github.com/apache/airflow/actions/runs/28635348211 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
