allthingssecurity commented on PR #28256: URL: https://github.com/apache/camel/pull/28256#issuecomment-6334637510
Thanks Claus. Pushed 6c1cb79e635b: 1. The comment now says what reading `lastTime` once is for (your wording), and the test comment no longer calls it a shutdown. The PR description no longer gives the shutdown case as the reason: `doShutdown` closes and clears the client first, as you say. 2. Late events: agreed. I wrote a follow-up JIRA draft for Shashank to file, to bring the look-back lookup of CAMEL-25727 over to this task (with the cost per check against the `LookupEvents` limit). camel-aws-secrets-manager: 12 tests, 0 failures. _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]
