geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jgenen...@apache.org
Subject svn commit: r264928 - in /geronimo/trunk/modules: jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java
Date Wed, 31 Aug 2005 03:02:11 GMT
Author: jgenender
Date: Tue Aug 30 20:02:07 2005
New Revision: 264928

URL: http://svn.apache.org/viewcvs?rev=264928&view=rev
Log:
Fix for GERONIMO-484 for logging memory leak

Modified:
    geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
    geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java

Modified: geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java?rev=264928&r1=264927&r2=264928&view=diff
==============================================================================
--- geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
(original)
+++ geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Tue Aug 30 20:02:07 2005
@@ -352,6 +352,10 @@
             leaveContextScope(null, null, context);
             jettyContainer.removeContext(this);
         }
+
+        // No more logging will occur for this ClassLoader. Inform the LogFactory to avoid
a memory leak.
+        LogFactory.release(webClassLoader);
+        
         log.info("JettyWebAppContext stopped");
     }
 

Modified: geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java?rev=264928&r1=264927&r2=264928&view=diff
==============================================================================
--- geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java
(original)
+++ geronimo/trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatWebAppContext.java
Tue Aug 30 20:02:07 2005
@@ -379,11 +379,17 @@
     public void doStop() throws Exception {
         container.removeContext(this);
 
+        // No more logging will occur for this ClassLoader. Inform the LogFactory to avoid
a memory leak.
+        LogFactory.release(webClassLoader);
+
         log.info("TomcatWebAppContext stopped");
     }
 
     public void doFail() {
         container.removeContext(this);
+
+        // No more logging will occur for this ClassLoader. Inform the LogFactory to avoid
a memory leak.
+        LogFactory.release(webClassLoader);
 
         log.info("TomcatWebAppContext failed");
     }



Mime
View raw message