]
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: