camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashwin Karpe (JIRA)" <>
Subject [jira] Commented: (CAMEL-3575) allow ScheduledRoutePolicy to handle more than one action (start, stop, resume, pause)
Date Thu, 27 Jan 2011 15:42:43 GMT


Ashwin Karpe commented on CAMEL-3575:

Thanks to Ricardo Veguilla for the patch.

Fix applied along with a unit test and removal of an unnecessary Routebuilder import in ScheduledJobState.

Committed to the Camel trunk as revision r1064154.



> allow ScheduledRoutePolicy to handle more than one action (start, stop, resume, pause)
> --------------------------------------------------------------------------------------
>                 Key: CAMEL-3575
>                 URL:
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-quartz
>    Affects Versions: 2.6.0
>            Reporter: Ricardo Veguilla
>            Assignee: Ashwin Karpe
>             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.

View raw message