Author: markt
Date: Fri Mar 27 13:20:46 2009
New Revision: 759130

URL: http://svn.apache.org/viewvc?rev=759130&view=rev
Log:
Withdraw my patch. Vote for Filip's.

Modified:
    tomcat/current/tc5.5.x/STATUS.txt

Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=759130&r1=759129&r2=759130&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Fri Mar 27 13:20:46 2009
@@ -98,11 +98,6 @@
   +1: markt, rjung
   -1: 
 
-* Fix thread safety issues in date formats
-  http://svn.apache.org/viewvc?view=rev&revision=708160
-  +1: markt, rjung
-  -1: fhanik - the date tool contention patch later on takes care of this, and 
makes it concurrent, this will still have lock contention and require more 
patches to fix the same issue.
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46408
   Correct potential invalid cast
   http://svn.apache.org/viewvc?rev=728032&view=rev
@@ -140,13 +135,8 @@
 * Lock contention during cookie creation, implementation is single threaded
   http://people.apache.org/~fhanik/tomcat/datetool-lock-contention.patch
   This patch realizes that DateFormat is not thread safe, but uses them 
without the need for synchronization using thread locals
-  +1: fhanik
+  +1: fhanik, markt
   -1: 
-  rjung: http://svn.apache.org/viewvc?rev=708160&view=rev above seems
-  to already fix all cases, in which the problematic DateTool code was
-  actually used in TC 5.5.x.
-  Either we should use Mark's patch (removing DateTool) or yours
-  (making DateTool performing better).
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46606
   Make max depth configurable for WebDAV servlet



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

Reply via email to