Rangsh commented on PR #12218: URL: https://github.com/apache/seatunnel/pull/12218#issuecomment-5653038136
@DanielLeens @SEZ9 Rebased onto latest `dev` as requested (head `6ed996701e`). While rebasing, adapted the rollback onto the post-#12238 executionId / context-keyed deployment model rather than reintroducing location-keyed `cancellationFutures`: - Keep two-phase publish (`cancellationFutures` / async / timer maps by `TaskGroupContext`, then `executionContexts` last) - On post-publish failure: abort tracker, drain cooperative queue, generation-safe `executionContexts` remove, clear context-keyed maps, release owned classloaders - `BlockingWorker` / `CooperativeTaskWorker` early guards retained - Both-sides regression tests updated for `executionId` + context-keyed cancellation assertions; redeploy still goes through `deployTask(Data)` Local: `TaskExecutionServiceTest` — `Tests run: 20, Failures: 0, Errors: 0, Skipped: 0` PR is now `MERGEABLE` against `dev`. Remaining MinIO pull failures (if they still appear) remain the pre-existing Docker Hub infra issue tracked by #12287 and are unrelated to this diff. -- 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]
