geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xiam...@apache.org
Subject svn commit: r1326524 - /geronimo/server/branches/3.0-beta/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
Date Mon, 16 Apr 2012 07:53:07 GMT
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;
                                 }
                             }



Mime
View raw message