search

jlinn / quartz-redis-jobstore

A Quartz Scheduler JobStore using Redis.
Apache License 2.0
136 stars 67 forks source link

scheduledFireTime is not set correctly when trigger fires #68

Open k1th opened 2 years ago

k1th commented 2 years ago 
            TriggerFiredBundle triggerFiredBundle = new TriggerFiredBundle(job, trigger, calendar, false, new Date(), previousFireTime, previousFireTime, trigger.getNextFireTime());

sets scheduledFireTime not to the "real" scheduled time when misfires are processed. This plays a role when the misfire policy is set to MISFIRE_INSTRUCTION_IGNORE_MISFIRE_POLICY

Fix: Change to 

            TriggerFiredBundle triggerFiredBundle = new TriggerFiredBundle(job, trigger, calendar, false, new Date(), trigger.getPreviousFireTime(), previousFireTime, trigger.getNextFireTime());

like the other job stores do.