You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/07/14 23:39:40 UTC

svn commit: r964216 - in /tomcat/trunk: java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java webapps/docs/changelog.xml

Author: markt
Date: Wed Jul 14 21:39:40 2010
New Revision: 964216

URL: http://svn.apache.org/viewvc?rev=964216&view=rev
Log:
Fxi https://issues.apache.org/bugzilla/show_bug.cgi?id=49127
Don't swallow exceptions unnecessarily

Modified:
    tomcat/trunk/java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java
    tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java?rev=964216&r1=964215&r2=964216&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java (original)
+++ tomcat/trunk/java/org/apache/catalina/ha/session/SimpleTcpReplicationManager.java Wed Jul 14 21:39:40 2010
@@ -520,7 +520,7 @@ public class SimpleTcpReplicationManager
                 do {
                     try {
                         Thread.sleep(100);
-                    }catch ( Exception sleep) {}
+                    } catch (InterruptedException sleep) { /* Ignore */ }
                     reqNow = System.currentTimeMillis();
                     isTimeout=((reqNow-reqStart)>(1000*60));
                 } while ( (!isStateTransferred()) && (!isTimeout));

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=964216&r1=964215&r2=964216&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Wed Jul 14 21:39:40 2010
@@ -190,6 +190,10 @@
   <subsection name="Cluster">
     <changelog>
       <fix>
+        <bug>49127</bug>: Don't swallow exceptions unnecessarily in
+        <code>SimpleTcpReplicationManager.startInternal()</code>. (markt)
+      </fix>
+      <fix>
         <bug>49445</bug>: When session ID is changed after authentication,
         ensure the DeltaManager replicates the change in ID to the other nodes
         in the cluster. (kfujino)



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