You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ri...@apache.org on 2010/09/07 13:56:59 UTC

svn commit: r993323 - /geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java

Author: rickmcguire
Date: Tue Sep  7 11:56:59 2010
New Revision: 993323

URL: http://svn.apache.org/viewvc?rev=993323&view=rev
Log:
Don't wrap EJBExceptions in another EJBException

Modified:
    geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java

Modified: geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java
URL: http://svn.apache.org/viewvc/geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java?rev=993323&r1=993322&r2=993323&view=diff
==============================================================================
--- geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java (original)
+++ geronimo/specs/trunk/geronimo-ejb_3.1_spec/src/main/java/javax/ejb/embeddable/EJBContainer.java Tue Sep  7 11:56:59 2010
@@ -59,6 +59,10 @@ public abstract class EJBContainer {
                 }
             }
             throw new EJBException("Provider error. No provider definition found");
+        } catch (EJBException e) {
+            // if the container provider throws an EJBException, don't wrap another one 
+            // around the original. 
+            throw e;
         } catch (Exception e) {
             throw new EJBException("Provider error. No provider found", e);
         }