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