SEZ9 commented on PR #12079: URL: https://github.com/apache/seatunnel/pull/12079#issuecomment-6621530224
Thanks for the detailed rundown against `67f3bde2506a`. I will re-check all of the open points (F1 through F8) directly against the diff at `67f3bde2506a`, including the tests you listed for F1, F2 and F4 and the shared `TransformDependencyScheduler` for F3, rather than asking for further confirmation here. I'd prefer to confirm F5, F6 and F7 from the diff itself as well before marking them resolved. I will update my review based on what the code and tests actually assert, and if anything still looks off after that, I will post a concrete config that reproduces it. <!-- streview-comment:2263 --> -- 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]
