The GitHub Actions job "Required Checks" on texera.git/main has succeeded. Run started by GitHub user github-merge-queue[bot] (triggered by github-merge-queue[bot]).
Head commit for run: 425f5a5ad94ed600f6aa0ccdce29f366b17b586d / Meng Wang <[email protected]> refactor(amber): remove the unused SuccessExecutionResult DTO (#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) Report URL: https://github.com/apache/texera/actions/runs/32561647739 With regards, GitHub Actions via GitBox
