wgzhao commented on PR #12904:
URL: https://github.com/apache/seatunnel/pull/12904#issuecomment-6283811226
Thanks for the detailed review. I addressed the remaining items:
- Documented `found: false` in the deprecated v2 `/running-job/:jobId`
blocks and in both v1
blocks (EN/ZH), which share `JobInfoService`.
- CLI polling now retries one ambiguous response. A repeated status-less
body gets a distinct
response-shape/failover message, and the post-submit path no longer claims
the id may be wrong.
- The Web UI now catches a rejected `getJobInfo` and renders an error result
instead of a blank
page; the missing-job card also shows the requested job id.
- `JobsApi.job_exists` raises `TypeError` for a non-JSON body instead of
returning `False`.
Validation on the new head `3908e3bf1`: server
`JobInfoServiceNullSafetyTest` 8 tests, UI
`detail.spec.ts` 4 tests plus `vue-tsc`, Python SDK 11 tests, CLI
`test_diagnose_cli.py` 29
tests; `spotless:check` and the engine-server `verify` pass.
--
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]
