Ma77Ball opened a new pull request, #5390:
URL: https://github.com/apache/texera/pull/5390

   ### What changes were proposed in this PR?
   - Pin the exact binary-preview formatting in `convertTuplesToJson`: the 
existing spec asserted only byte size and the presence of an ellipsis, so this 
adds the 13-bits-or-fewer branch (a 1-byte field renders its full bit string 
with no ellipsis) and the exact leading-10 + ellipsis + trailing-3 format for 
larger fields.
   - Add serialization-contract tests for the `WebOutputMode` / 
`WebResultUpdate` ADTs the frontend dispatches on: the `type` discriminator for 
`PaginationMode`/`SetSnapshotMode`/`SetDeltaMode`, plus the shapes of 
`WebPaginationUpdate` and `WebDataUpdate`.
   ### Any related issues, documentation, or discussions?
   Closes: #5389
   ### How was this PR tested?
   - Run `sbt "WorkflowExecutionService/testOnly *ExecutionResultServiceSpec"` 
and expect 16 examples to pass (11 existing, 5 new).
   - Local sbt test runs are blocked by JaCoCo 0.8.11 (cannot instrument Java 
25 bytecode, major version 69), so the suite was run directly via ScalaTest off 
the dependency classpath; all 16 passed. CI runs it on a supported JDK.
   ### Was this PR authored or co-authored using generative AI tooling?
   Co-authored with Claude Opus 4.8 in compliance with ASF


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to