tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kkoli...@apache.org
Subject svn commit: r794917 - /tomcat/current/tc5.5.x/STATUS.txt
Date Fri, 17 Jul 2009 00:25:48 GMT
Author: kkolinko
Date: Fri Jul 17 00:25:48 2009
New Revision: 794917

URL: http://svn.apache.org/viewvc?rev=794917&view=rev
Log:
vote

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=794917&r1=794916&r2=794917&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Fri Jul 17 00:25:48 2009
@@ -113,4 +113,10 @@
   clusters
   http://svn.apache.org/viewvc?rev=794822&view=rev
   +1: markt, rjung
+  +1: kkolinko: I like the patch, but I do not like the jvmRoute.noCluster
+    message text. It is informational, but that "You must use.." sounds
+    like a warning. Maybe rephrase it to something more delicate, e.g.:
+     The JvmRouterBinderValve is configured, but clustering is not being used. In this case
you have to use a PersistentManager for fail-over.
+    As a side note: the class Javadoc comment for JvmRouteBinderValve is
+    hard to read because of grammatical errors...
   -1: 



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


Mime
View raw message