geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xiam...@apache.org
Subject svn commit: r1326527 - /geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
Date Mon, 16 Apr 2012 07:53:58 GMT
Author: xiaming
Date: Mon Apr 16 07:53:57 2012
New Revision: 1326527

URL: http://svn.apache.org/viewvc?rev=1326527&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, for trunk

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

Modified: geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java?rev=1326527&r1=1326526&r2=1326527&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
(original)
+++ geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/EmbeddedDaemon.java
Mon Apr 16 07:53:57 2012
@@ -293,7 +293,11 @@ public class EmbeddedDaemon {
                                 } 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