geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From manugeo...@apache.org
Subject svn commit: r722150 - /geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
Date Mon, 01 Dec 2008 17:57:36 GMT
Author: manugeorge
Date: Mon Dec  1 09:57:35 2008
New Revision: 722150

URL: http://svn.apache.org/viewvc?rev=722150&view=rev
Log:
GERONIMO-4178 Misleading errors during deployment when missing entries in openejb-jar.xml.
Captures and throws a DeploymentException if the containerId is not set as the RA is not specified

Modified:
    geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java

Modified: geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java?rev=722150&r1=722149&r2=722150&view=diff
==============================================================================
--- geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
(original)
+++ geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
Mon Dec  1 09:57:35 2008
@@ -873,6 +873,11 @@
             // add a dependency from the module to the ra so we can be assured the mdb
             // container exists when this app is started
             ejbModuleGBeanData.addDependency(resourceAdapterAbstractName);
+            for(MessageDrivenBeanInfo mdbInfo:mdbs.values()){
+                if(mdbInfo != null && mdbInfo.containerId == null){
+                    throw new DeploymentException("No Resource Adapter defined for MDB '"
+ mdbInfo.ejbName + "'");
+                }
+            }
         }
     }
 



Mime
View raw message