[
https://issues.apache.org/jira/browse/CAMEL-8955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14623923#comment-14623923
]
Claus Ibsen commented on CAMEL-8955:
------------------------------------
Read the EIP patterns, and see that the aggregator has options of on completion
and whatnot. The enrich is only using agg strategy to merge the source with the
received target - not to group N+ together and have different completion
triggers and whatnot. All that latter is only for the aggregator EIP.
> Processor for .pollEnrich incorrectly handles provided AggregationStrategy
> --------------------------------------------------------------------------
>
> Key: CAMEL-8955
> URL: https://issues.apache.org/jira/browse/CAMEL-8955
> Project: Camel
> Issue Type: Bug
> Components: camel-core
> Affects Versions: 2.14.3
> Reporter: Andy Fedotov
> Assignee: Claus Ibsen
> Priority: Minor
> Attachments: PollEnrichAggregationStrategyTest.java
>
>
> As it can be concluded from AggregationStrategy javadoc, and also from
> available implementations such as GroupedExchangeAggregationStrategy, it
> should be used in the following manner:
> {code:java}
> AggregationStrategy strategy = new GroupedExchangeAggregationStrategy();
> Exchange result = null;
> result = strategy.aggregate(result, exchange1);
> result = strategy.aggregate(result, exchange2);
> // ... and so on
> return result;
> {code}
> But PollEnricher do it in following manner:
> {code:java}
> AggregationStrategy strategy = new GroupedExchangeAggregationStrategy();
> Exchange result = strategy.aggregate(exchange1, exchange2);
> return result;
> {code}
> This leads to incorrect result returned after aggregation and lost exchanges.
> Also PollEnricher doesn't handle CompletionAwareAggregationStrategy and does
> not call onCompletion(). Consequently GroupedExchangeAggregationStrategy does
> not return aggregated result in the Exchange body.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
