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-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to