DakshinJeeva opened a new pull request, #29505: URL: https://github.com/apache/camel/pull/29505
Fixes CAMEL-26395. ### Problem In `RedisAggregationRepository`, when optimistic removal is used with recovery enabled, the active exchange was removed before being persisted in the recovery map. If the recovery write failed, the exchange could be lost from both maps. ### Solution - Persist the exchange in the recovery map before removing it from the active map. - If optimistic removal fails, clean up the recovery entry before throwing `OptimisticLockingException`. - Add integration tests covering recovery-write failure and optimistic-locking failure. ### Tests - `RedisAggregationRepositoryOperationsIT` - `./mvnw -pl components/camel-redis -Psourcecheck clean install` - All 42 Redis module tests passed. ### AI assistance ChatGPT (OpenAI) assisted with understanding the issue, implementation, and regression tests. I reviewed the changes and ran the tests and source checks. -- 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]
