svn commit: r1608231 - /tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java

[rmannibucau]

Author: rmannibucau
Date: Sun Jul  6 16:01:44 2014
New Revision: 1608231

URL: http://svn.apache.org/r1608231
Log:
logging if we skipped or not app classloader creation

Modified:
    
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java

Modified: 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
URL: 
http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java?rev=1608231&r1=1608230&r2=1608231&view=diff
==============================================================================
--- 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
 (original)
+++ 
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
 Sun Jul  6 16:01:44 2014
@@ -1935,7 +1935,7 @@ public class Assembler extends Assembler
         // try to get the app BM from the AppClassLoader having stored it in a 
map).
         // since we don't really need to create a classloader here when 
starting from classpath just let skip this step
         if (skipLoaderIfPossible) { // TODO: maybe use a boolean to know if 
all urls comes from the classpath to avoid this validation
-            final Collection<File> urls = new ArrayList<File>();
+            final Collection<File> urls = new HashSet<>();
             for (final URL url : ClassLoaders.findUrls(parent)) { // need to 
convert it to file since urls can be file:/xxx or jar:file:///xxx
                 try {
                     urls.add(URLs.toFile(url));
@@ -1958,10 +1958,13 @@ public class Assembler extends Assembler
             }
 
             if (allIsIntheClasspath) {
+                logger.info("Not creating another application classloader for 
" + appInfo.appId);
                 return parent;
             }
         }
 
+        logger.info("Creating dedicated application classloader for " + 
appInfo.appId);
+
         if (!appInfo.delegateFirst) {
             return ClassLoaderUtil.createClassLoader(appInfo.path, filtered, 
parent);
         }