Issue Details (XML | Word | Printable)

Key: APF-872
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Won't Fix
Priority: Trivial Trivial
Assignee: Matt Raible
Reporter: Matt Raible
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
AppFuse

Add LogFactory.release() for Commons Logging

Created: 05/Sep/07 01:13 AM   Updated: 14/Sep/07 04:22 PM   Resolved: 14/Sep/07 04:22 PM
Component/s: None
Affects Version/s: 2.0-RC1
Fix Version/s: 2.0 Final


 Description  « Hide
Suggested in a comment on my blog:

http://raibledesigns.com/rd/entry/why_i_like_tomcat_5#comment9

Use LogFactory.release(Thread.currentThread().getContextClassLoader()) in your ServletContextListener's contextDestroyed() method to release the resources it takes.

2927 by  Matt Raible (1 file)
05/Sep/07 11:17 PM (35 months, 7 days ago)
Matt Raible made changes - 05/Sep/07 11:18 PM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Matt Raible added a comment - 14/Sep/07 04:22 PM
Reopened to mark "won't fix" because of the following warnings issued by SLF4J when it's in the classpath (caused by displaytag upgrade).

WARN: The method class org.apache.commons.logging.impl.SLF4JLogFactory#release() was invoked.
WARN: Please see http://www.slf4j.org/codes.html for an explanation.

Matt Raible made changes - 14/Sep/07 04:22 PM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Matt Raible made changes - 14/Sep/07 04:22 PM
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Won't Fix [ 2 ]
2949 by  Matt Raible (1 file)
14/Sep/07 04:23 PM (34 months, 30 days ago)
APF-872: Reverted call to release() because of the following warnings issued by SLF4J when it's in the classpath (caused by displaytag upgrade).

WARN: The method class org.apache.commons.logging.impl.SLF4JLogFactory#release() was invoked.
WARN: Please see http://www.slf4j.org/codes.html for an explanation.
appfuse: trunk/web/common/src/main/java/org/appfuse/webapp/listener/StartupListener.java 2949 history download (+5 -2) diffs