You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2011/07/22 01:58:42 UTC
svn commit: r1149408 - in
/geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config:
DebugLoggingLifecycleMonitor.java SimpleConfigurationManager.java
Author: djencks
Date: Thu Jul 21 23:58:41 2011
New Revision: 1149408
URL: http://svn.apache.org/viewvc?rev=1149408&view=rev
Log:
slightly better logging
Modified:
geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/DebugLoggingLifecycleMonitor.java
geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/SimpleConfigurationManager.java
Modified: geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/DebugLoggingLifecycleMonitor.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/DebugLoggingLifecycleMonitor.java?rev=1149408&r1=1149407&r2=1149408&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/DebugLoggingLifecycleMonitor.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/DebugLoggingLifecycleMonitor.java Thu Jul 21 23:58:41 2011
@@ -67,7 +67,7 @@ public class DebugLoggingLifecycleMonito
}
public void failed(Artifact configurationId, Throwable cause) {
- log.debug("failed module: {}", configurationId);
+ log.debug("failed module: {}", configurationId, cause);
}
public void finished() {
Modified: geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/SimpleConfigurationManager.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/SimpleConfigurationManager.java?rev=1149408&r1=1149407&r2=1149408&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/SimpleConfigurationManager.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/SimpleConfigurationManager.java Thu Jul 21 23:58:41 2011
@@ -696,7 +696,7 @@ public class SimpleConfigurationManager
configurationId = unstartedConfiguration;
Configuration configuration = getConfiguration(configurationId);
if (configuration == null) {
- throw new NoSuchConfigException(configurationId, "trying to start ancestor config, but not found");
+ throw new NoSuchConfigException(configurationId, "trying to start ancestor config for config " + id + ", but not found");
}
monitor.starting(configurationId);
start(configuration);