DanielLeens commented on PR #12355: URL: https://github.com/apache/seatunnel/pull/12355#issuecomment-5775122849
Thanks for pulling the actual job log instead of just the aggregate status - that confirms my suspicion. My review stands: no code-side blockers, approval recorded on the current head, and this CI failure does not implicate connector-cdc-mysql or anything this PR touches. For anyone following along later: I re-checked the fork's `Build` run on this same head just now, and the failing set has since rotated (`unit-test (8, windows-latest)` and `doris-connector-it (8, ubuntu-latest)`), not the openGauss job you diagnosed a few days ago. I pulled both logs - the Windows failure is `PayPalClientTest.closeWakesRetryWait`, the same pre-existing timing flake in `connector-http-paypal` noted in earlier rounds, and the Doris failure is `DorisSchemaChangeIT`, an unrelated Doris sink E2E error. Neither touches this PR's files or `connector-cdc-mysql` in any way, so the picture is unchanged even though the specific red job moved around - this looks like normal CI noise rotating across unrelated suites rather than anything tied to this fix. So from my side there is nothing left to do here: the fix and its tests are sound, and this is purely waiting on a committer with write access to re-approve `e6e9b6b3193ec9c25397849c24b11a177394a5cf` so branch protection is satisfied. I'll keep watching and will flag promptly if anything changes. -- 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]
