geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gdam...@apache.org
Subject svn commit: r106703 - /geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java
Date Sat, 27 Nov 2004 10:13:42 GMT
Author: gdamour
Date: Sat Nov 27 02:13:41 2004
New Revision: 106703

URL: http://svn.apache.org/viewcvs?view=rev&rev=106703
Log:
A configuration can be loaded by the Kernel and not running. In such a case,
it is not possible to retrieve its configurationClassLoader attribute as it
is not a special attribute.

In some specific situations, where the parent Configuration was loaded and not 
running, a DeploymentException was raised as the configurationClassLoader
could not be retrieved.

Note that this was working until now as the attribute previously retrieved
was classLoader, which is a special attribute.

Modified:
   geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java

Modified: geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java
Url: http://svn.apache.org/viewcvs/geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java?view=diff&rev=106703&p1=geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java&r1=106702&p2=geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java&r2=106703
==============================================================================
--- geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java
(original)
+++ geronimo/trunk/modules/deployment/src/java/org/apache/geronimo/deployment/DeploymentContext.java
Sat Nov 27 02:13:41 2004
@@ -51,6 +51,7 @@
 import org.apache.geronimo.gbean.jmx.GBeanMBean;
 import org.apache.geronimo.gbean.GBeanData;
 import org.apache.geronimo.kernel.Kernel;
+import org.apache.geronimo.kernel.management.State;
 import org.apache.geronimo.kernel.repository.Repository;
 import org.apache.geronimo.kernel.config.Configuration;
 import org.apache.geronimo.kernel.config.ConfigurationManager;
@@ -115,16 +116,20 @@
                 throw new DeploymentException("Unable to load parents", e);
             }
 
-            for (Iterator i = ancestors.iterator(); i.hasNext();) {
-                ObjectName name = (ObjectName) i.next();
-                try {
-                    // start the config to get the classloaders going,
-                    // by not specfying startRecursive none of the GBeans should start
-                    kernel.startGBean(name);
-                } catch (Exception e) {
-                    throw new DeploymentException(e);
+            try {
+                ObjectName currentConfig = parentName;
+                while ( State.RUNNING != State.fromInteger((Integer) kernel.getAttribute(currentConfig,
"state")) ) {
+                    kernel.startGBean(currentConfig);
+                    URI currentParentID = (URI) kernel.getAttribute(currentConfig, "parentID");
+                    if ( null == currentParentID ) {
+                        break;
+                    }
+                    currentConfig = Configuration.getConfigurationObjectName(currentParentID);
                 }
+            } catch (Exception e) {
+                throw new DeploymentException(e);
             }
+                
             try {
                 parentCL = (ClassLoader) kernel.getAttribute(parentName, "configurationClassLoader");
             } catch (Exception e) {

Mime
View raw message