DanielLeens commented on PR #12595: URL: https://github.com/apache/seatunnel/pull/12595#issuecomment-6158012320
Thanks @SEPURI-SAI-KRISHNA, the Flink 1.20.1 mapping and the root-cause chain are very helpful, and thanks for correcting the JDK 8 point in the javadoc. One concrete next step that should clear the red `Build`: the direct-memory failure you traced in #12757 now has a fix on `dev`. #12756 (`[Fix][E2E] Prevent direct memory exhaustion in Flink 1.20 Cassandra tests`, merge commit `07d6f2235a`) was merged at 2026-10-13 16:03 UTC. It only touches `Flink20Container.java` in `seatunnel-e2e-common`, so it does not interact with this PR's diff. Your head `364514f12c` (committed 2026-10-11) does not contain it yet. So my earlier "a rerun cannot clear this while `dev` is red" no longer holds. Please merge the latest `dev` into this branch (this one time it is useful, not just because the branch is behind) and let `Build` run again. The `CassandraIT` failures on the Flink 1.20.1 container should then go away. If `all-connectors-it-6` (`DatabendCDCSinkIT`) is still red afterwards, that one is independent of this change and a single `--failed` rerun is the right tool. I have nothing further on the code. This is a static review comment; I did not execute the PR. -- 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]
