[#APF-872] Add LogFactory.release() for Commons Logging - AppFuse JIRA

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.

Sort Order:

2927 by Matt Raible (1 file)

05/Sep/07 11:17 PM (35 months, 7 days ago)

~~APF-872~~: Add LogFactory.release() for Commons Logging

http://www.nabble.com/LogFactory.release%28%29-for-Commons-Logging--tf4214401s2369.html#a11989686

appfuse: trunk/web/common/src/main/java/org/appfuse/webapp/listener/StartupListener.java 2927

(+2 -2) diffs

Matt Raible made changes - 05/Sep/07 11:18 PM

Field	Original Value	New Value
Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

[ Permalink | « Hide ]

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

(+5 -2) diffs