DanielLeens commented on PR #12298: URL: https://github.com/apache/seatunnel/pull/12298#issuecomment-5701047912
Thanks for going back and re-checking the per-leg numbers, @SEZ9 — the corrected picture (JDK 11 only, intermittent rather than deterministic, and not reproduced consistently on #12299's JDK 11 legs either) is a more honest read of the evidence than the earlier "well-supported hypothesis" framing, and I'd rather have that correction on the record than let the stronger claim stand uncontested. Agreed this changes nothing about #12298 itself: `NebulaGraphIT` exercises the Nebula Graph connector's container bootstrap, `OpengaussCDCIT` exercises Opengauss CDC restore, and neither is reachable from a change confined to `JettyService`'s constructor and the `HttpConfig.port` field. Filing #12344 and #12345 to track them independently of this PR is the right call — that keeps both failure patterns visible and investigable without gating this change on unrelated flakes. My earlier assessment stands: no source-level objections, this PR's own test passes, and the two red jobs are now both tracked as pre-existing/environmental issues rather than something this diff introduces. Still ready to merge once a maintainer with merge rights picks it up. -- 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]
