svn commit: r371450 - /jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java

[sk...@apache.org]

Author: skitching
Date: Sun Jan 22 19:18:47 2006
New Revision: 371450

URL: http://svn.apache.org/viewcvs?rev=371450&view=rev
Log:
Update javadoc now that WeakHashtable is part of the standard distribution.
ServletContextCleaner is only a backup for odd corner cases.

Modified:
    jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java

Modified: jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java?rev=371450&r1=371449&r2=371450&view=diff
==============================================================================
--- jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java (original)
+++ jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/ServletContextCleaner.java Sun Jan 22 19:18:47 2006
@@ -31,23 +31,19 @@
  * a webapp, and responds by ensuring that commons-logging releases all
  * memory associated with the undeployed webapp.
  * <p>
- * This class is necessary when code using commons-logging is deployed within
- * a servlet container and commons-logging is deployed via a shared classloader.
- * When commons-logging is deployed in a "shared" classloader that is visible 
- * to all webapps, commons-logging still ensures that each webapp gets its own
- * logging configuration by setting up a separate LogFactory object for each 
- * context classloader, and ensuring that the correct LogFactory object is 
- * used based on the context classloader set whenever LogFactory.getLog is 
- * called. However the negative side of this is that LogFactory needs to keep 
- * a static map of LogFactory objects keyed by context classloader; when the 
- * webapp is "undeployed" this means there is still a reference to the 
- * undeployed classloader preventing the memory used by all its classes from 
- * being reclaimed.  
+ * In general, the WeakHashtable support added in commons-logging release 1.1
+ * ensures that logging classes do not hold references that prevent an
+ * undeployed webapp's memory from being garbage-collected even when multiple
+ * copies of commons-logging are deployed via multiple classloaders (a
+ * situation that earlier versions had problems with). However there are
+ * some rare cases where the WeakHashtable approach does not work; in these
+ * situations specifying this class as a listener for the web application will
+ * ensure that all references held by commons-logging are fully released.
  * <p>
  * To use this class, configure the webapp deployment descriptor to call
  * this class on webapp undeploy; the contextDestroyed method will tell
- * LogFactory that the entry in its map for the current webapp's context
- * classloader should be cleared.
+ * every accessable LogFactory class that the entry in its map for the
+ * current webapp's context classloader should be cleared.
  * 
  * @since 1.1
  */



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org