camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christian Schneider (JIRA)" <>
Subject [jira] Commented: (CAMEL-2517) AggregateGroupedExchangeMultipleCorrelationTest fails
Date Wed, 03 Mar 2010 21:55:44 GMT


Christian Schneider commented on CAMEL-2517:

Hi Claus,

I am currently trying to understand the code in Aggregator. Though I canĀ“t  say I fully understand
everything I stumbled over something interesting.
In line 488 there is the following command:
timeoutMap = new AggregationTimeoutMap(scheduler, 1000L);

Could this be the reason why my timeout of 1000ms caused the success? If I understand this
correctly the map only checks every 1000ms. So as the test by default only delays the last
exchange for 100ms it is not sure which group times out first. When I lower the delay in line
488 to below 100ms the test works again. So I guess a valid fix could be to set the sleep
to 1500ms and the completionTimeout to 2000ms. 

What do you think?



> AggregateGroupedExchangeMultipleCorrelationTest fails
> -----------------------------------------------------
>                 Key: CAMEL-2517
>                 URL:
>             Project: Apache Camel
>          Issue Type: Test
>          Components: camel-core
>    Affects Versions: 2.2.0
>         Environment: Windows Vista
>            Reporter: Christian Schneider
>             Fix For: 2.3.0
>         Attachments: camel-core-2010-01-02.patch, failure_100ms.txt, success_1000ms.txt
> On my system the test AggregateGroupedExchangeMultipleCorrelationTest fails. It seems
the Thread.sleep(100) is too short.
> When I increase to at least 900ms it works. To add some safety I propose to increase
to 1000ms.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message