The GitHub Actions job "Required Checks" on texera.git/fix/5042-logicallink-operatoridentity-round-trip has failed. Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).
Head commit for run: ed1b32bfd153077e06b90865bb2d4ce369c345dd / William Wong <[email protected]> fix(workflow-compiling-service): allow LogicalLink OperatorIdentity round-trip Prior to this change, `@JsonCreator` was placed on the `String` convenience constructor in both `amber` and `workflow-compiling-service` `LogicalLink`. Jackson serialises `OperatorIdentity` as an object (`{"id":"op-A"}`), so any round-trip through `writeValueAsString` / `readValue` failed with `MismatchedInputException` because the String constructor cannot accept an object node. Fix: add a private `readOperatorIdentity(JsonNode, String)` helper to the companion object and move `@JsonCreator` to a new `JsonNode` constructor that delegates to it. The helper handles the string shape (front-end input), the object shape (serialised form), null/absent fields, and rejects non-text / non-object nodes with `IllegalArgumentException`. The new `workflow-compiling-service` `LogicalLinkSpec` adds direct unit coverage of the `readOperatorIdentity` branches that the existing HTTP-layer-only `WorkflowCompilationResourceSpec` did not reach. It also pins the leniency contract (no `require` guards) and the Jackson annotation contract (`@JsonProperty` key pinning, round-trip fidelity). The `amber` `LogicalLinkSpec` is updated to match the renamed constructor section, drop the now-invalid MismatchedInputException expectation, and add the round-trip test. Closes #5042 Report URL: https://github.com/apache/texera/actions/runs/27667773940 With regards, GitHub Actions via GitBox
