SEZ9 commented on PR #12079: URL: https://github.com/apache/seatunnel/pull/12079#issuecomment-6282856107
Thanks for the detailed re-check against head `612b5ede517b5f7b800c2695c8e2f3e88a789b97`, and for noting that the `Build` check is reported as `SUCCESS` for run 39008501172 on `612b5ede5`. The locations you listed for F3 (single resolution path in the `ScheduledTransform` constructor), F7 (the `Transform Dependency Resolution` entries in both the en and zh incompatible-changes docs) and F8 (production callers of `getOutputId()` and removal of the static helper) are exactly what I needed. I'll verify them directly against the diff on this head rather than re-requesting anything, and will mark the three items resolved once that matches up. No further questions from my side for now; I'll do the final read-through on `612b5ede5` and follow up here. <!-- streview-comment:1899 --> -- 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]
