DanielLeens commented on PR #10340: URL: https://github.com/apache/seatunnel/pull/10340#issuecomment-6473965593
Thanks for the contribution. The merge-gate fact cited in my previous conclusion has changed: GitHub now reports a real merge conflict. This follow-up corrects that gate on the unchanged reviewed head; it is not another full source review or a sync request based only on being behind. Gate snapshot (2026-10-28T16:00:53+00:00), head `59e5bec4a356de208ae48e588a26a00f9cf87edb`: [Build](https://github.com/apache/seatunnel/runs/121484682451) is `FAILURE`. The blocking jobs are [Run / Dead links](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120704388210) `failure`; [Run / updated-modules-integration-test-part-2 (17, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657532) `failure`; [Run / updated-modules-integration-test-part-1 (11, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657546) `failure`; [Run / unit-test (17, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657577) `failure`; [Run / updated-modules-integration-test-part-1 (17, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657590) `failure`; [Run / updated-modules-integration-test-part-4 (11, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657596) `failure`; [Run / updated-modules-integration-test-part-4 (17, ubuntu-latest)](https://github.com/dsome han/seatunnel/actions/runs/40038706513/job/120706657598) `failure`; [Run / updated-modules-integration-test-part-2 (11, ubuntu-latest)](https://github.com/dsomehan/seatunnel/actions/runs/40038706513/job/120706657603) `failure`; plus 33 other failed/cancelled jobs in the linked run. Observed log evidence: `Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.10.1:compile (default-compile) on project connector-jdbc: Compilation failure` Please inspect or rerun only the named failed job where a rerun is appropriate. The logs establish the failing stage, but do not alone establish whether the current PR caused it. No matching upstream cure has been proved by this follow-up; a base sync alone is not a verified resolution, so no unrelated module/dependency patch is requested. My [previous source conclusion](https://github.com/apache/seatunnel/pull/10340#pullrequestreview-5635862149) remains unchanged, including its independent source/documentation requests. CI is not being passed by inference, and this follow-up is not an effective approval. Separately, GitHub reports merge conflicts with `dev`. Please sync the latest base and resolve the conflicts first; the resulting new head needs checks and re-review. This conflict gate is independent of CI and existing source findings. -- 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]
