DanielLeens commented on PR #11626: URL: https://github.com/apache/seatunnel/pull/11626#issuecomment-5678698814
Thanks @davidzollo! This is exactly the piece I flagged as missing in my last status update — my own approvals here are comment-only for this repo's branch protection, so they never counted toward the required review, and this PR has been sitting at `mergeStateStatus: BLOCKED`/`REVIEW_REQUIRED` purely on that gate rather than on any open correctness concern. Nothing has changed on the code side since my last full pass on this same head (`ff980aca9`, 2026-08-31): the one High-severity blocker (the `assertNoRunningJobs()` Awaitility wrapper not actually retrying) is fixed, and the two remaining items are both Low-severity polish, not blockers. `reviewDecision` now shows `APPROVED` with `Build` green on this head, so from where I sit this is ready to merge whenever you'd like to take it in. @goutamadwant — congrats, this was a long and thorough round-trip and the shared-container contract is in much better shape for it. Appreciate you sticking with every follow-up. -- 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]
