allthingssecurity commented on PR #28196: URL: https://github.com/apache/camel/pull/28196#issuecomment-6313947402
Thanks Andrea, all three are done in 04278d365217. 1. `doStart` now creates the `PubSubReconnectTask` scheduler only when there is none and no longer shuts it down on a restart; `doShutdown` still does. A loop of the previous start that waits in the retry delay is then still woken up and ends on the generation check, and a throttled route no longer gets a new scheduler on every cycle. 2. `asyncPullLoopOfPreviousStartEndsWhenConsumerIsRestarted` now verifies that the consumer executor of the first start is shut down through the `ExecutorServiceManager`, and that the scheduler is created once (that second check fails with the previous commit: `TooManyActualInvocations`). 3. The test class is package-private. camel-google-pubsub unit tests: 28 run, 0 failures. The description is updated. _Claude Code on behalf of allthingssecurity_ -- 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]
