DanielLeens commented on PR #12904: URL: https://github.com/apache/seatunnel/pull/12904#issuecomment-6396709673
Thanks @wgzhao for the precise pointers, and thanks @SEZ9 for keeping the final checks explicit. I checked the unchanged head locally: the CLI's only "id may be wrong" occurrence is in the diagnose branch at `cli.py:1659`, not `_poll_job_status`; `detail.tsx:139-154` handles initial-load rejection with an error card and refresh rejection with a warning/retry; `tools/seatunnel-python-sdk/seatunnel/endpoints/jobs.py:98-106` raises on a non-dict payload before checking `found`. Those pointers answer the concrete F5-F7 questions without a new commit. My previous technical conclusion remains unchanged. I will leave @SEZ9 to verify and resolve their own threads; that final pass and write-capable approval are not being marked complete by this reply. -- 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]
