[jboss-cvs] JBossAS SVN: r64024 - trunk/server/src/main/org/jboss/ejb/txtimer.

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

Author: scott.stark at jboss.org
Date: 2007-07-13 00:53:26 -0400 (Fri, 13 Jul 2007)
New Revision: 64024

Modified:
   trunk/server/src/main/org/jboss/ejb/txtimer/TimerServiceImpl.java
Log:
JBAS-4053, thrown an EJBException if the create fails

Modified: trunk/server/src/main/org/jboss/ejb/txtimer/TimerServiceImpl.java
===================================================================
--- trunk/server/src/main/org/jboss/ejb/txtimer/TimerServiceImpl.java	2007-07-13 04:45:56 UTC (rev 64023)
+++ trunk/server/src/main/org/jboss/ejb/txtimer/TimerServiceImpl.java	2007-07-13 04:53:26 UTC (rev 64024)
@@ -242,7 +242,8 @@
     *                                  a state that does not allow access to this method.
     * @throws javax.ejb.EJBException   If this method could not complete due to a system-level failure.
     */
-   public Timer createTimer(Date initialExpiration, long intervalDuration, Serializable info) throws IllegalArgumentException, IllegalStateException, EJBException
+   public Timer createTimer(Date initialExpiration, long intervalDuration, Serializable info)
+      throws IllegalArgumentException, IllegalStateException, EJBException
    {
       if (initialExpiration == null)
          throw new IllegalArgumentException("initial expiration is null");
@@ -259,11 +260,10 @@
       }
       catch (Exception e)
       {
-         log.error("Cannot create txtimer", e);
-         return null;
+         throw new EJBException("Failed to create timer", e);
       }
    }
-   
+
    /**
     * Get all the active timers associated with this bean.
     *