Rangsh commented on issue #12164: URL: https://github.com/apache/seatunnel/issues/12164#issuecomment-5612322230
@SEZ9 Thanks — the four review asks are already spelled out in the PR description checklist on https://github.com/apache/seatunnel/pull/12218 (rollback boundary, deterministic failure injection + real second deploy, master-failover skip preservation, and scope on current `dev`). A follow-up commit also landed there to address DanielLeens' recommended Issue 2 (`CooperativeTaskWorker` null/isCancel guard symmetric with `BlockingWorker`). Remaining discussion will stay on the PR as you suggested. -- 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]
