[Springframework-cvs] spring/src/org/springframework/scheduling/quartz SchedulerFactoryBean.java, 1.44, 1.45

[Juergen H. <jho...@us...>]

Update of /cvsroot/springframework/spring/src/org/springframework/scheduling/quartz
In directory fdv4jf1.ch3.sourceforge.com:/tmp/cvs-serv8681/src/org/springframework/scheduling/quartz

Modified Files:
	SchedulerFactoryBean.java 
Log Message:
added explicit test for Scheduler isolation per ApplicationContext

Index: SchedulerFactoryBean.java
===================================================================
RCS file: /cvsroot/springframework/spring/src/org/springframework/scheduling/quartz/SchedulerFactoryBean.java,v
retrieving revision 1.44
retrieving revision 1.45
diff -C2 -d -r1.44 -r1.45
*** SchedulerFactoryBean.java	20 Oct 2008 14:00:46 -0000	1.44
--- SchedulerFactoryBean.java	27 Oct 2008 13:43:17 -0000	1.45
***************
*** 68,73 ****
   * <p>When using persistent jobs, it is strongly recommended to perform all
   * operations on the Scheduler within Spring-managed (or plain JTA) transactions.
!  * Else, database locking will not properly work and might even break.
!  * (See {@link #setDataSource setDataSource} javadoc for details.)
   *
   * <p>The preferred way to achieve transactional execution is to demarcate
--- 68,73 ----
   * <p>When using persistent jobs, it is strongly recommended to perform all
   * operations on the Scheduler within Spring-managed (or plain JTA) transactions.
!  * Otherwise, database locking will not properly work and might even break.
!  * (See the {@link #setDataSource setDataSource} javadoc for details.)
   *
   * <p>The preferred way to achieve transactional execution is to demarcate
***************
*** 82,90 ****
   * @author Juergen Hoeller
   * @since 18.02.2004
   * @see #setDataSource
   * @see org.quartz.Scheduler
   * @see org.quartz.SchedulerFactory
   * @see org.quartz.impl.StdSchedulerFactory
!  * @see org.springframework.transaction.interceptor.TransactionProxyFactoryBean
   */
  public class SchedulerFactoryBean extends SchedulerAccessor
--- 82,93 ----
   * @author Juergen Hoeller
   * @since 18.02.2004
+  * @see #setConfigLocation
+  * @see #setQuartzProperties
   * @see #setDataSource
+  * @see #setJobFactory
   * @see org.quartz.Scheduler
   * @see org.quartz.SchedulerFactory
   * @see org.quartz.impl.StdSchedulerFactory
!  * @see org.quartz.Job
   */
  public class SchedulerFactoryBean extends SchedulerAccessor
***************
*** 576,580 ****
  				}
  				if (!this.exposeSchedulerInRepository) {
! 					// Need to remove it in this case, since Quartz shares the Scheduler instance by default!
  					SchedulerRepository.getInstance().remove(newScheduler.getSchedulerName());
  				}
--- 579,584 ----
  				}
  				if (!this.exposeSchedulerInRepository) {
! 					// Need to explicitly remove it if not intended for exposure,
! 					// since Quartz shares the Scheduler instance by default!
  					SchedulerRepository.getInstance().remove(newScheduler.getSchedulerName());
  				}