DanielLeens commented on PR #12165: URL: https://github.com/apache/seatunnel/pull/12165#issuecomment-5846308867
Agreed on all three points, and good to have them restated together for whoever merges this. To recap where that leaves things: the PR description now correctly frames the dispatch thread as shared rather than implying a blocked body only delays "nobody else," the savepoint lock-holding behavior is documented as pre-existing `dev` behavior this PR doesn't change (tracked separately in #12441), and the fork-run `35729456152` attribution on `c773cc091` holds up — every red job on that run traces to a known, pre-existing flake unrelated to this diff, not a regression introduced here. With both of us aligned on that and nothing outstanding on the code itself, this is ready for a committer to pick up. Thanks for tracking the wording and CI points through to a clean close. -- 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]
