SEZ9 commented on PR #12079: URL: https://github.com/apache/seatunnel/pull/12079#issuecomment-6104828518
Thanks for syncing with `dev`. I re-read the full diff on the new head `ad8cadd12` rather than only the merge delta. The PR-owned source files are unchanged between `7bb568104` and `ad8cadd12`; the only differences are in the two `incompatible-changes.md` files, and those are just other entries that landed on `dev`, with the "Transform Dependency Resolution" entry still present in both EN and ZH. On this head the earlier points still hold as resolved: the runtime parser, the graph pre-check and the dry-run validator share one `TransformDependencyScheduler`; duplicate default outputs release a waiting join exactly once; an explicit self-reference ends in a concrete cycle error instead of looping; and the compatibility change is intentional and documented in the incompatible-changes notes. My earlier conclusion stands and I have no new source-side concern from this sync. <!-- streview-comment:1666 --> -- 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]
