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 2006/05/08 02:18:38 UTC

svn commit: r404880 - /geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java

Author: djencks
Date: Sun May  7 17:18:35 2006
New Revision: 404880

URL: http://svn.apache.org/viewcvs?rev=404880&view=rev
Log:
Improve logging of classloader structure

Modified:
    geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java

Modified: geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java?rev=404880&r1=404879&r2=404880&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java (original)
+++ geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/Configuration.java Sun May  7 17:18:35 2006
@@ -85,7 +85,6 @@
  */
 public class Configuration implements GBeanLifecycle, ConfigurationParent {
     private static final Log log = LogFactory.getLog(Configuration.class);
-    public static final Object JSR77_BASE_NAME_PROPERTY = "org.apache.geronimo.name.javax.management.j2ee.BaseName";
 
     /**
      * Converts an Artifact to an AbstractName for a configuration.  Does not
@@ -351,7 +350,20 @@
         }
         String[] nonOverridableClasses = (String[]) nonOverridableSet.toArray(new String[nonOverridableSet.size()]);
 
-        log.debug("ClassPath for " + id + " resolved to " + Arrays.asList(urls));
+        if (log.isDebugEnabled()) {
+            StringBuffer buf = new StringBuffer("ClassLoader structure for configuration ").append(id).append("\n");
+            buf.append("Parent configurations:\n");
+            for (Iterator iterator = classParents.iterator(); iterator.hasNext();) {
+                Configuration configuration = (Configuration) iterator.next();
+                buf.append("     ").append(configuration.getId()).append("\n");
+            }
+            buf.append("ClassPath:\n");
+            for (int i = 0; i < urls.length; i++) {
+                URL url = urls[i];
+                buf.append("     ").append(url).append("\n");
+            }
+            log.debug(buf.toString());
+        }
 
         if (Boolean.getBoolean("Xorg.apache.geronimo.OldClassLoader")) {
             return new MultiParentClassLoader(environment.getConfigId(),