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

   ## Summary
   
   _Claude Code on behalf of davsclaus_
   
   - Move the `UseOriginalAggregationStrategy` binding from `Splitter` and 
`RecipientListProcessor` into `MulticastProcessor` (their shared base class), 
so all three EIPs consistently bind the original exchange on the strategy.
   - Previously Multicast never called `newInstance(exchange)`, making the 
strategy silently ineffective — especially in error scenarios where the 
aggregated result could overwrite the original exchange body instead of 
preserving it.
   - Remove the now-redundant duplicate binding blocks from `Splitter` and 
`RecipientListProcessor` (Splitter's null-strategy fallback for its default 
behavior is preserved).
   
   ## Test plan
   
   - [x] Added body-preservation assertion to 
`MulticastUseOriginalPropagateExceptionCaughtTest`
   - [x] All 18 `UseOriginal*` tests pass across Multicast, Splitter, and 
RecipientList
   - [x] Full `camel-core` test suite passes (7264 tests, 0 failures)
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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