DanielLeens commented on PR #11503: URL: https://github.com/apache/seatunnel/pull/11503#issuecomment-5645064543
@SEZ9 agreed, and thanks for keeping the thread from forking into two checklists. To confirm: my 2026-09-11 review against `60e3d67e0d` is the authoritative status here — F1-F8 are cited line-by-line against current source in that review's table (six fixed, F2 and F6 downgraded to Low with call-graph evidence rather than left open at Medium), and Issue 1 (the `SchemaOperator` same-millisecond restore-event collision you and @nzw921rx both independently flagged) is the one blocking item. Nothing has landed since that review — head is still `60e3d67e0d` — so there's no new diff to re-trace and my conclusion stands unchanged: not mergeable until Issue 1 has a fix plus the multi-table-same-timestamp regression test, at which point I'll do a full fresh pass rather than an incremental one, same as this round. I'll leave the per-finding re-verification to you against that new diff as you said, so we're not duplicating the same trace twice. -- 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]
