[CAMEL-15244] AggregationStrategy - default timeout method should be empty

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.3, 3.5.0
• Component/s: camel-core
• Labels: help-wanted
• Estimated Complexity: Novice
• Assignee: Andrea Cosentino
• Reporter: Claus Ibsen
• Created: 25/Jun/20 15:46
• Updated: 20/Jul/20 06:33
• Resolved: 20/Jul/20 06:33

Description

As reported on gitter

Hi, I've noticed we get the following warning when using groupedBodyAggregationStrategy with sjms-batch.

2020-06-25 22:49:19.522 WARN ||| 21068 — [msBatchConsumer] o.a.c.p.a.GroupedBodyAggregationStrategy : Parallel processing timed out after 500 millis for number 18. This task will be cancelled and will not be aggregated.
I think it comes from this default interface implementation on AggregationStrategy.

default void timeout(Exchange exchange, int index, int total, long timeout) {
// log a WARN we timed out since it will not be aggregated and the Exchange will be lost
LoggerFactory.getLogger(getClass()).warn("Parallel processing timed out after {} millis for number {}. This task will be cancelled and will not be aggregated.", timeout, index);
}
I think the warning is wrong as the aggregated messages do seem to come through. It is just annoying in the logs. I can change log level for this category to error but I'm concerned there may be genuine info that will be hidden. Any thoughts?

Issue Links

• links to: GitHub Pull Request #4020
• links to: GitHub Pull Request #4021