https://issues.apache.org/bugzilla/show_bug.cgi?id=49943

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

--- Comment #7 from Mark Thomas <ma...@apache.org> 2010-10-03 19:31:39 EDT ---
The patch as provided does not work. It fails to take account of the caching of
resources that occurs in the web application class loaders. I'm also curious
how the re-reading of the CATALINA_BASE/conf/logging.properties was being
triggered.

Having reviewed the patch in more detail there are also a few places where
unnecessary cosmetic changes are made. This makes the patch harder to review.

I'm marking this as an enhancement and would be happy to revisit this to review
a patch that addresses this issues. The patch will be applied to 7.0.x before
it is applied to 6.0.x so patches against that code base are preferred.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to