You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ma...@apache.org on 2008/12/01 18:57:36 UTC
svn commit: r722150 -
/geronimo/server/branches/2.1/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
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 + "'");
+ }
+ }
}
}