Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Message-ID: <1011776654.4751267653344630.JavaMail.jira@brutus.apache.org>
Date: Wed, 3 Mar 2010 21:55:44 +0000 (UTC)
From: "Christian Schneider (JIRA)" <jira@apache.org>
To: camel-dev@activemq.apache.org
Subject: [jira] Commented: (CAMEL-2517)
 AggregateGroupedExchangeMultipleCorrelationTest fails
In-Reply-To: <1418278630.3171267565864431.JavaMail.jira@brutus.apache.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/activemq/browse/CAMEL-2517?page=3Dcom.atlas=
sian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D=
58007#action_58007 ]=20

Christian Schneider commented on CAMEL-2517:
--------------------------------------------

Hi Claus,

I am currently trying to understand the code in Aggregator. Though I can=C2=
=B4t  say I fully understand everything I stumbled over something interesti=
ng.
In line 488 there is the following command:
timeoutMap =3D 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 gro=
up times out first. When I lower the delay in line 488 to below 100ms the t=
est works again. So I guess a valid fix could be to set the sleep to 1500ms=
 and the completionTimeout to 2000ms.=20

What do you think?

Greetings

Christian


> AggregateGroupedExchangeMultipleCorrelationTest fails
> -----------------------------------------------------
>
>                 Key: CAMEL-2517
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2517
>             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, succ=
ess_1000ms.txt
>
>
> On my system the test AggregateGroupedExchangeMultipleCorrelationTest fai=
ls. 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.

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