[
https://issues.jboss.org/browse/SEAMCRON-11?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Peter Royle resolved SEAMCRON-11.
---------------------------------
Resolution: Done
Last part was to modify the underlying method of scheduling @Every jobs.
Instead of using cron expressions which can't properly handle for example
minute intervals which are not factors of 60, we're using a hand-built trigger
with the proper repeat interval set.
> Add java.util.concurrent.TimeUnit attribute on @Every
> -----------------------------------------------------
>
> Key: SEAMCRON-11
> URL: https://issues.jboss.org/browse/SEAMCRON-11
> Project: Seam Cron
> Issue Type: Enhancement
> Reporter: Peter Royle
> Assignee: Peter Royle
> Fix For: 3.0.0.Alpha1
>
>
> Idea courtesy of George Gastaldi. This would allow us to replace Second,
> Minute and Hour with the standard CronEvent for better consistency, without
> breaking the readability.
> public void doStuff(@Observes @Every(SECONDS) CronEvent
> Unfortunately there's a couple of teeny tiny hiccups:
> - TimeUnit only goes up to SECONDS in JDK 1.5, and I want to support 1.5 so
> that Seam Cron can be used n the various CDI JavaSE flavors.
> - TimeUnit values are pluralised which don't read very well, eg: @Observes
> @Every(HOURS)
> So obviously, we just create our own replacement for TimeUnit. I guess it
> will have to be ... CronTimeUnit?
> I should also note that this flies in the face of the possibility of
> implementing something like:
> @Observes @Every @Nth(5) Minute m
> We could maybe do something like this instead?:
> @Observes @Every(nth=5, value=MINUTE) CronEvent e
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues
