CODEWITHSUNDEEP
javaquartz-scheduler
Adder
Adder

Reputation: 5868

Java Quartz scheduler trigger nextFireTime is null

I am creating a Quartz trigger for various schedule frequencies. At the end of the function I return the trigger, but debug output it before the return statement. The debug output is this Trigger 'DEFAULT.6da64b5bd2ee-91b0fa99-6e11-4356-a55f-e80353b61fc4': triggerClass: 'org.quartz.impl.triggers.DailyTimeIntervalTriggerImpl calendar: 'null' misfireInstruction: -1 nextFireTime: null. I am worried about nextFireTime being null, will the trigger still fire?

Part of the code looks like this:

if (freqType.equalsIgnoreCase(KpiDefinition.KPI_FREQ_TYPE_DAILY)
                || freqType
                        .equalsIgnoreCase(KpiDefinition.KPI_FREQ_TYPE_SCHEDULED))
        {
            DailyTimeIntervalScheduleBuilder schedule = DailyTimeIntervalScheduleBuilder
                    .dailyTimeIntervalSchedule()
                    .startingDailyAt(startTime)
                    .endingDailyAt(endTime)
                    .onEveryDay()
                    .withInterval(kpiDef.getKpiFrequency().intValue(),
                            IntervalUnit.valueOf(kpiDef.getKpiFreqTimeUnit()))
                    .withMisfireHandlingInstructionIgnoreMisfires();

            Date startDate = kpiDef.getKpiStartDate();
            if (startDate.before(new Date()))
            {
                startDate = new Date();
            }

            if (freqType.equalsIgnoreCase(KpiDefinition.KPI_FREQ_TYPE_DAILY))
            {

                trigger = newTrigger().withSchedule(schedule)
                        .startAt(startDate).build();
            } else if (freqType
                    .equalsIgnoreCase(KpiDefinition.KPI_FREQ_TYPE_SCHEDULED))
            {
                Date endDate = kpiDef.getKpiEndDate();

                // This means that schedule has already passed and so KPI should
                // not be scheduled.
                if (endDate.before(new Date()))
                {
                    logger.debug("getTriggerWithSchedule for KPI " + kpiDef.getKpiDefId() + " null Schedule returned for end date " +endDate.toString());
                    return null;
                }
                trigger = newTrigger().withSchedule(schedule)
                        .startAt(startDate).endAt(endDate).build();
            }
        }

Upvotes: 0

Views: 3184

Answers (1)

walen
walen

Reputation: 7273

nextFireTime is only resolved for CronTriggers in my experience. SimpleTrigger for example also has nextFireTime=null. So I wouldn't worry.

Bear in mind also this:

The value returned is not guaranteed to be valid until after the Trigger has been added to the scheduler.

http://www.quartz-scheduler.org/api/2.1.7/org/quartz/Trigger.html#getNextFireTime()

Upvotes: 1

Related Questions