abdessalems commented on PR #11727: URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5659338078
@SEZ9 — both are already up: the test Javadoc update is in 01a705240, and the PR description was updated the same day. Ready for your pass whenever you have time. One correction to my earlier note on F7, so you don't find it as a surprise: none of the ownership-internals reflection remains, but TaskDeployStaleContextRaceTest does still reflect into executionContexts (line 276) — that's how the remover thread simulates an entry disappearing mid-deploy. The TaskExecutionServiceTest additions don't use reflection. Given your earlier call on F7 to keep reflection rather than widen the production API, I've left it as is. -- 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]
