DanielLeens commented on PR #11503: URL: https://github.com/apache/seatunnel/pull/11503#issuecomment-5437623287
Thanks for the clear handoff summary, @davidzollo — that lines up with what I already confirmed in the approval above. No new commit since `1e5493c9f8`, so there's nothing further for me to re-review at this point. On CI: my independent read of the run matches your classification — the 3 non-green jobs (Maven Central connection reset on `testcontainers:databend`, the known `CouchbaseIT` container-bootstrap flake, and the 120-minute budget cancellation mid-passing-tests) are all infra-related and unrelated to this diff. I have no objection to that read. @SEZ9 — from my side, the 2026-08-24 NPE blocker is fixed and regression-covered on the current head, and I independently re-verified the `RestoreTableSchemaEvent` contract Javadoc as part of the approval. Would appreciate your fresh pass on the current head whenever you get a chance, so the standing `CHANGES_REQUESTED` can be resolved one way or the other — happy to dig into any of PR11503-F1..F8 further if something still doesn't sit right after re-checking. -- 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]
