SchedulerFactoryBean should support non-durable jobs as well [SPR-11689]

[spring-projects-issues]

Dzmitry Lazerka opened SPR-11689 and commented

Reproducible with both Quartz 2.1.7 and the most recent 2.2.1.

Examples of configs are:

1. http://forum.spring.io/forum/spring-projects/container/7915-problem-with-quartz-and-spring-durable-jobs
2. Our config:

<bean name="ImportJob" class="org.springframework.scheduling.quartz.JobDetailFactoryBean">
    <property name="jobClass" value="my.ImportJob"/>
</bean>
<bean id="importTrigger" class="org.springframework.scheduling.quartz.SimpleTriggerFactoryBean">
    <property name="jobDetail" ref="ImportJob"/>
    <property name="group" value="etl"/>
    <property name="startDelay" value="10000"/>
    <property name="repeatInterval" value="${import.interval}"/>
</bean>
<bean name="scheduler" class="org.springframework.scheduling.quartz.SchedulerFactoryBean">
    <property name="schedulerName" value="Our Scheduler"/>
    <property name="autoStartup" value="false" />
    <property name="waitForJobsToCompleteOnShutdown" value="true"/>
    <property name="triggers">
        <list>
            <ref bean="importTrigger"/>
        </list>
    </property>
</bean>

Quick debugging showed that org.springframework.scheduling.quartz.SchedulerAccessor adds jobDetail without its trigger, although it has the trigger.

---

Affects: 4.0.3

Reference URL: http://forum.spring.io/forum/spring-projects/container/7915-problem-with-quartz-and-spring-durable-jobs

Referenced from: commits 9a59292

[spring-projects-issues]

Juergen Hoeller commented

Indeed, historically Spring-defined Quartz jobs effectively had to be defined as durable (setting their "durability" property to "true") when passed to SchedulerFactoryBean's job/trigger registration properties. I suppose your scenario works as well once the job definition has been marked as durable?

We revised our Quartz support for Spring Framework 4.1 already (making it Quartz 2.1+ now), so we'll revise this job registration part along with it: not requiring job definitions to be durable when they are associated with a trigger at registration time.

Juergen