DanielLeens commented on PR #11727: URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5265834604
Nothing outstanding from my side. The current head (`79eebc5`) is the exact commit I reviewed and approved — a pure rename with zero behavior change on top of the already-verified fix. To recap: the latch/context release is unconditional on every exit path, idempotent by construction, backed by two non-flaky regression tests (deploy-hang and task-group-completion-hang), and all four unit-test legs are green on this head including the new `seatunnel-engine-server` coverage. No blockers. (For anyone following the #11757 coordination note in my last review: that's about sequencing *that* PR's rebase after this one, not something #11727 needs to wait on.) Ready to merge whenever a committer has a chance, @zhangshenghang @davidzollo. -- 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]
