You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xi...@apache.org on 2012/04/16 09:53:07 UTC

svn commit: r1326524 - /geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java

Author: xiaming
Date: Mon Apr 16 07:53:07 2012
New Revision: 1326524

URL: http://svn.apache.org/viewvc?rev=1326524&view=rev
Log:
GERONIMO-5802 Revise the patch not to set load=false for failed module automatically, instead let user decide what to do next

Modified:
    geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java

Modified: geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java?rev=1326524&r1=1326523&r2=1326524&view=diff
==============================================================================
--- geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java (original)
+++ geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java Mon Apr 16 07:53:07 2012
@@ -242,7 +242,11 @@ public class EmbeddedDaemon implements M
                             } catch (Exception e) {
                                 if ( configModuleType != ConfigurationModuleType.SERVICE.getValue() ) {
                                     log.warn("Failed to start module " + configID + "; Cause by " + e.getCause());
-                                    configurationManager.unloadConfiguration(configID);
+                                    log.warn("Please try to correct the problem by referring to the logged exception. " +
+                                    		"If you want to bypass it in future restart, you can set load=\"false\" on " +
+                                    		"that module.");
+                                    // Only log warning and let user decide what action to take
+                                    //configurationManager.unloadConfiguration(configID);
                                     continue;
                                 }
                             }