mengw15 opened a new pull request, #7836:
URL: https://github.com/apache/texera/pull/7836
### What changes were proposed in this PR?
Deletes `SuccessExecutionResult` and its spec — 68 lines, no behaviour
change. The class arrived unreferenced in the 2020 amber bulk import
(`41a8a9201`) and mirrors the `{ code, resultID, result }` response envelope
that #1214 removed from the frontend in 2021; nothing in `amber`, `frontend`,
`agent-service`, or any resource/config references it, so its only consumer was
its own coverage spec, which goes with it. Not to be confused with
`SyncExecutionResult` in the same package, which is live.
### Any related issues, documentation, discussions?
Closes #7829.
### How was this PR tested?
`WorkflowExecutionService/Test/compile` passes after the deletion — the
compiler confirms nothing referenced the class; a repo-wide text search finds
no other occurrence.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (claude-fable-5)
--
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]
