davidzollo commented on PR #11077: URL: https://github.com/apache/seatunnel/pull/11077#issuecomment-5549179581
## Descoped this branch back to just the multi-table-sink fix Per the outstanding scope note, I pushed `975962f` which removes `TaskExecutionService.java` — it had drifted in through a `dev`-sync merge and doesn't belong here. It's a stale, earlier snapshot of the `BlockingWorker` start-latch fix already tracked and more thoroughly reviewed under #11727 (which has since evolved to use a per-generation `ownedContext` instead of a shared `executionContexts` lookup). No new PR needed for it — it lands via #11727. The file was checked out verbatim from the current `dev` HEAD, so this reverts exactly that one out-of-scope diff and nothing else. The multi-table-sink diff itself (`MultiTableSink`, `MultiTableSinkWriter`, `MultiTableSinkAggregatedCommitter`, `MultiTableSinkCommitter`, tests, docs) is untouched — same content already reviewed as solid across many rounds. @DanielLeens — over to you for a fresh CI/diff check on the cleaned head. -- 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]
