SEZ9 commented on PR #11503: URL: https://github.com/apache/seatunnel/pull/11503#issuecomment-5643160083
Quick follow-up to keep this thread tidy rather than re-posting the eight findings from the earlier pass on `f77cdfa677d4` as open items. The full re-review against the current head (`60e3d67e0d`) already supersedes that pass: it carries first-hand, line-level citations for F1–F8 and formalizes the restore-event timestamp-collision issue in `SchemaOperator` as blocking. Please treat that review as the authoritative list, so this comment isn't taken as a separate or conflicting checklist. Once a new head lands that addresses the blocking issue, I'll re-check F1–F8 directly against the diff and update the status of each finding from the code, so there's no need to summarize the changes in comments. Thanks for bearing with the long thread. <!-- streview-comment:989 --> -- 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]
