Author: markt
Date: Tue Apr 20 22:29:24 2010
New Revision: 936093
URL: http://svn.apache.org/viewvc?rev=936093&view=rev
Log:
Vote
Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL:
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=936093&r1=936092&r2=936093&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Apr 20 22:29:24 2010
@@ -102,7 +102,7 @@ PATCHES PROPOSED TO BACKPORT:
Additional patch:
http://svn.apache.org/viewvc?rev=935983&view=rev
- +1: kkolinko
+ +1: kkolinko, markt
-1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48729
@@ -251,7 +251,7 @@ PATCHES PROPOSED TO BACKPORT:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49116
If session is already invalid, expire session to prevent memory leak.
http://svn.apache.org/viewvc?view=revision&revision=933881
- +1: kfujino
+ +1: kfujino, markt
-1:
* sessionCounter and expiredSessions declares as long instead of int.
@@ -259,6 +259,7 @@ PATCHES PROPOSED TO BACKPORT:
+1: kfujino
-1: kkolinko: I think that we cannot change o.a.c.Manager API in such way
in a dot release. Alternative proposal below.
+ -1: markt: We can't change the API in 6. It will have to wait until 7.
* Fix possible overflows when calculating session statistics
Fixes a) miscalculating the "average" because of multiplication overflow,
@@ -266,11 +267,13 @@ PATCHES PROPOSED TO BACKPORT:
The fix for trunk is already provided by r934337.
http://people.apache.org/~kkolinko/patches/2010-04-17_tc6_StandardSession_statistics.patch
+1: kkolinko
+ +0: markt You can still update the average, even if you can't update the
+ count
-1:
* Improve the ChatServlet comet example and fix some issues there
http://svn.apache.org/viewvc?rev=935105&view=rev
- +1: kkolinko
+ +1: kkolinko, markt
-1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49151
@@ -278,11 +281,11 @@ PATCHES PROPOSED TO BACKPORT:
http://svn.apache.org/viewvc?view=revision&revision=935510
+1: kfujino
+1: kkolinko: and also +1 for
http://svn.apache.org/viewvc?rev=935777&view=rev
- -1:
+ -1: markt: and also +1 for http://svn.apache.org/viewvc?rev=935777&view=rev
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49158
http://svn.apache.org/viewvc?view=revision&revision=935998
- +1: fhanik
+ +1: fhanik, markt
-1:
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
