You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by jl...@apache.org on 2012/12/07 23:43:47 UTC

svn commit: r1418530 - /openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java

Author: jlmonteiro
Date: Fri Dec  7 22:43:47 2012
New Revision: 1418530

URL: http://svn.apache.org/viewvc?rev=1418530&view=rev
Log:
Add a fallback to the parent classloader for Atmosphere mainly

Modified:
    openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java

Modified: openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java?rev=1418530&r1=1418529&r2=1418530&view=diff
==============================================================================
--- openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java (original)
+++ openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java Fri Dec  7 22:43:47 2012
@@ -87,7 +87,11 @@ public class LazyStopWebappClassLoader e
 
         // avoid to redefine classes from server in this classloader is it not already loaded
         if (URLClassLoaderFirst.shouldSkip(name)) {
-            return OpenEJB.class.getClassLoader().loadClass(name);
+            try {
+                return OpenEJB.class.getClassLoader().loadClass(name);
+            } catch (ClassNotFoundException e) {
+                return super.loadClass(name);
+            }
         }
         return super.loadClass(name);
     }



Fwd: svn commit: r1418530 - /openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java

Posted by Romain Manni-Bucau <rm...@gmail.com>.
Think we should catch noclassdeffounderror too,

Anyone to add it on trunk + branch please? Cant do it before monday
---------- Message transféré ----------
De : <jl...@apache.org>
Date : 7 déc. 2012 23:44
Objet : svn commit: r1418530 -
/openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
À : <co...@openejb.apache.org>

Author: jlmonteiro
Date: Fri Dec  7 22:43:47 2012
New Revision: 1418530

URL: http://svn.apache.org/viewvc?rev=1418530&view=rev
Log:
Add a fallback to the parent classloader for Atmosphere mainly

Modified:

openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java

Modified:
openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
URL:
http://svn.apache.org/viewvc/openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java?rev=1418530&r1=1418529&r2=1418530&view=diff
==============================================================================
---
openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
(original)
+++
openejb/trunk/openejb/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
Fri Dec  7 22:43:47 2012
@@ -87,7 +87,11 @@ public class LazyStopWebappClassLoader e

         // avoid to redefine classes from server in this classloader is it
not already loaded
         if (URLClassLoaderFirst.shouldSkip(name)) {
-            return OpenEJB.class.getClassLoader().loadClass(name);
+            try {
+                return OpenEJB.class.getClassLoader().loadClass(name);
+            } catch (ClassNotFoundException e) {
+                return super.loadClass(name);
+            }
         }
         return super.loadClass(name);
     }