Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Message-ID: <16709968.227741296083443764.JavaMail.jira@thor>
Date: Wed, 26 Jan 2011 18:10:43 -0500 (EST)
From: "Ashwin Karpe (JIRA)" <jira@apache.org>
To: camel-dev@activemq.apache.org
Subject: [jira] Commented: (CAMEL-3575) allow ScheduledRoutePolicy to handle
 more than one action (start, stop, resume, pause)
In-Reply-To: <15381098.113861295638663910.JavaMail.jira@thor>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/CAMEL-3575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12987278#action_12987278 ] 

Ashwin Karpe commented on CAMEL-3575:
-------------------------------------

Hi,

I am not sure that this is an issue... It is indeed possible to set a start and stop time using a single policy for a given route. The only difference is that it sets two separate triggers to handle the start and stop.

Also it seems like the patch is missing unit tests to validate that the patch works...

There is indeed a problem in being able to apply multiple policies (policy lists on a given route in CAMEL-3254).

I will be working on this over the next few days and will apply a fix for this and CAMEL-3254 in 2.7.0, following the release of 2.6.0

Cheers,

Ashwin...

P.S: Please hold off on applying the above patch...


> allow ScheduledRoutePolicy to handle more than one action (start, stop, resume, pause)
> --------------------------------------------------------------------------------------
>
>                 Key: CAMEL-3575
>                 URL: https://issues.apache.org/jira/browse/CAMEL-3575
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-quartz
>    Affects Versions: 2.6.0
>            Reporter: Ricardo Veguilla
>             Fix For: 2.7.0
>
>         Attachments: ScheduledRoutePolicy-Patch.diff
>
>
> The current implementation of ScheuledRoutePolicy only supports one type of action (start,stop,resume,pause) for only one route.  If, for example,  you attempt to configure a CronScheduleRoutePolicy with a routeStartTime and routeStopTime, only the one will by handled because ScheduleRoutePolicy (super-class of CronScheduleRoutePolicy) only store one action and one route in the SchedulerContext:
>  protected void loadCallbackDataIntoSchedulerContext(Action action, Route route) throws SchedulerException {
>        getScheduler().getContext().put(SCHEDULED_ACTION, action);
>         getScheduler().getContext().put(SCHEDULED_ROUTE, route);
> }
> with the effect of creating two timers (one for startTime, the other for stopTime) that execute the same action (the last call to scheduleRoute).

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