Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-15244

AggregationStrategy - default timeout method should be empty

    XMLWordPrintableJSON

    Details

    • 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:
    • Estimated Complexity:
      Novice

      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?

        Attachments

          Activity

            People

            • Assignee:
              acosentino Andrea Cosentino
              Reporter:
              davsclaus Claus Ibsen
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: