[jboss-cvs] JBossAS SVN: r108163 - projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3.

[jboss-cvs-commits at lists.jboss.org]

Author: jaikiran
Date: 2010-09-16 04:42:51 -0400 (Thu, 16 Sep 2010)
New Revision: 108163

Modified:
   projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/TimerServiceContainer.java
Log:
EJBTHREE-2169 Move timerservice creation from lockedStart() to create()

Modified: projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/TimerServiceContainer.java
===================================================================
--- projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/TimerServiceContainer.java	2010-09-16 08:40:30 UTC (rev 108162)
+++ projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/TimerServiceContainer.java	2010-09-16 08:42:51 UTC (rev 108163)
@@ -92,36 +92,21 @@
     * {@link TimerService} creation is skipped for stateful session beans, since 
     * stateful session beans do not support timerservice
     * 
-    * @see EJBContainer#lockedStart()
+    * @see EJBContainer#create()
     */
    @Override
-   protected void lockedStart() throws Exception
+   public void create() throws Exception
    {
-      try
+      super.create();
+      // for non-stateful beans, create timer service
+      if (this.isStatefulBean() == false)
       {
-         super.lockedStart();
-         // for non-stateful beans, create timer service
-         if (this.isStatefulBean() == false)
-         {
-            // just create the timerservice. Restoring of
-            // any timers, will be done in afterStart(), once the container has fully started
-            // (to allow for timeout method invocations)
-            this.timerService = this.createTimerService();
-         }
-
+         // just create the timerservice. Restoring of
+         // any timers, will be done in afterStart(), once the container has fully started
+         // (to allow for timeout method invocations)
+         this.timerService = this.createTimerService();
       }
-      catch (Exception e)
-      {
-         try
-         {
-            this.lockedStop();
-         }
-         catch (Exception ignore)
-         {
-            logger.debug("Failed to cleanup after start() failure", ignore);
-         }
-         throw e;
-      }
+
    }
 
    /**