SEZ9 commented on PR #12218: URL: https://github.com/apache/seatunnel/pull/12218#issuecomment-6183425251
Thanks @Rangsh. Nothing new to add from my side since my last note: your reading of both failed jobs on fork run 38496659794 (head `35f5a2fcf7c`) matches mine, and the head has not moved since `154d6e9891a` apart from the clean `dev` merge, so my conclusion is unchanged. - `engine-v2-it (11)` (job 115650597710) is the known unrelated cancel-vs-worker-loss race in `SplitClusterFaultToleranceIT.testStreamJobCancelResolvesWhenWorkerCrashesBeforeCancelAck`; it does not go through a failed `deployLocalTask`, and a sync will not clear it. - `kafka-connector-it (17)` (job 115650597797) fails only in the Testcontainers image build (`received unexpected HTTP status: 500 Internal Server Error`) before any Kafka test logic runs, so I agree it is a runner/registry flake unrelated to this PR. No source change needed from you. Syncing with `dev` whenever convenient would give you a fresh fork run for the Kafka lane, but that is optional and not a gate from my side. The remaining step is the committer approval, which I cannot give as a read-only reviewer. Thanks again for the patient iterations. <!-- streview-comment:1763 --> -- 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]
