DanielLeens commented on PR #11971: URL: https://github.com/apache/seatunnel/pull/11971#issuecomment-5713031546
Thanks for tracking this down and posting the direct evidence, @yzeng1618 — that lines up exactly with what I found in my last review round: the `updated-modules-integration-test-part-4` failure is `OpengaussCDCIT#testAddFieldWithRestore`, a module this PR never touches. I independently checked #12344 and #12346: #12344 is open and confirms the same failure reproducing 40/40 times across two unrelated PRs (#12298, #12299) and two base commits, so this is a genuine `dev`-baseline flake, not something introduced by #11971. #12346 (the proposed fix) is also still open/unmerged, so there is nothing to sync against yet. Agreed on the plan: keep this PR focused and do not fold the CDC fix in here. Once #12346 (or whatever supersedes it) lands on `dev`, sync this branch and rerun `Build` — I would expect it to go green given this PR's own code, the `connector-jdbc` unit tests, and `JdbcMysqlTableOptionsIT` (7/7) are already passing on the current head (`6907e2f2f8fb`). No code blockers remain on my side; this is purely a CI-green gate plus final maintainer approval at this point. -- 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]
