DanielLeens commented on PR #12156: URL: https://github.com/apache/seatunnel/pull/12156#issuecomment-5601138648
Thanks @SEZ9 — agreed on all of it, nothing further from me. The blocking item stands as `cancelJob()` switching to the `jobStatus` local at line 212 (or an explicit PR-description callout if the null-tolerant fallthrough is intentional), with F2 left to the author's judgment on whether to fold into this PR or a follow-up, and F3/F4 as non-blocking doc/test cleanups. I'll re-review as soon as the F1 fix lands. -- 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]
