davsclaus opened a new pull request, #24821:
URL: https://github.com/apache/camel/pull/24821

   ## Summary
   
   - Add `try/catch(Exception)` guard to `AggregationIntervalTask.run()` and 
`RecoverTask.run()` so that a transient exception (e.g. repository hiccup) does 
not permanently cancel the scheduled background task
   - Make `aggregateCompletionCounter()` null-safe to avoid NPE when 
`CamelAggregatedCompletedBy` property is absent on recovered exchanges
   - Follows the same pattern already used by `DefaultTimeoutMap.purgeTask()` 
which catches and logs exceptions to keep the scheduled executor alive
   
   ## Test plan
   
   - [x] All 177 existing aggregator tests pass 
(`org.apache.camel.processor.aggregator.*Test`)
   - [ ] Verify with the reproducer tests attached to CAMEL-24141
   
   _Claude Code on behalf of davsclaus_
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


-- 
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]

Reply via email to