Fix broken cluster membership recovery

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1059346 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/java/org/apache/catalina/tribes/membership/McastServiceImpl.java b/java/org/apache/catalina/tribes/membership/McastServiceImpl.java
index c8aeca6..b6ef6c2 100644 (file)
--- a/java/org/apache/catalina/tribes/membership/McastServiceImpl.java
+++ b/java/org/apache/catalina/tribes/membership/McastServiceImpl.java
@@ -547,7 +547,7 @@ public class McastServiceImpl
                             try { Thread.sleep(500); } catch ( Exception ignore ){}
                             if ( (++errorCounter)>=recoveryCounter ) {
                                 errorCounter=0;
-                                new RecoveryThread(McastServiceImpl.this);
+                                new RecoveryThread(McastServiceImpl.this).start();
                             }
                         }
                     }
@@ -575,7 +575,7 @@ public class McastServiceImpl
                     else log.debug("Unable to send mcast message.",x);
                     if ( (++errorCounter)>=recoveryCounter ) {
                         errorCounter=0;
-                        new RecoveryThread(McastServiceImpl.this);
+                        new RecoveryThread(McastServiceImpl.this).start();
                     }
                 }
                 try { Thread.sleep(time); } catch ( Exception ignore ) {}

diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 73835bf..7f9ae99 100644 (file)
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -83,6 +83,11 @@
       <add>
         Internationalise the log messages for the FarmWarDeployer. (markt)
       </add>
+      <fix>
+        The recovery feature of the default cluster membership service can now
+        be enabled. Previously it any configuration settings for it were
+        ignored. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Web applications">

diff --git a/webapps/docs/config/cluster-membership.xml b/webapps/docs/config/cluster-membership.xml
index c477f29..1a623dd 100644 (file)
--- a/webapps/docs/config/cluster-membership.xml
+++ b/webapps/docs/config/cluster-membership.xml
@@ -138,8 +138,9 @@
     </attribute>
     <attribute name="recoveryCounter" required="false">
       <p>
-      When <code>recoveryEnabled==true</code> this value indicates how many times we will try recovery.
-      The default is <code>10</code>. <br/>
+      When <code>recoveryEnabled==true</code> this value indicates how many
+      times an error has to occur before recovery is attempted. The default is
+      <code>10</code>. <br/>
       </p>
     </attribute>
     <attribute name="recoverySleepTime" required="false">