Author: kfujino
Date: Wed Aug 23 06:24:21 2017
New Revision: 1805856

URL: http://svn.apache.org/viewvc?rev=1805856&view=rev
Log:
Avoid Ping timeout until the added map member by receiving MSG_START message is 
completely started.

Modified:
    
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
    tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java?rev=1805856&r1=1805855&r2=1805856&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
 Wed Aug 23 06:24:21 2017
@@ -281,12 +281,21 @@ public abstract class AbstractReplicated
                     MapMessage mapMsg = (MapMessage)resp[i].getMessage();
                     try {
                         mapMsg.deserialize(getExternalLoaders());
+                        Member member = resp[i].getSource();
                         State state = (State) mapMsg.getValue();
                         if (state.isAvailable()) {
-                            memberAlive(resp[i].getSource());
+                            memberAlive(member);
+                        } else if (state == State.STATETRANSFERRED) {
+                            synchronized (mapMembers) {
+                                if (log.isInfoEnabled())
+                                    log.info("Member[" + member + "] is state 
transferred but not available yet.");
+                                if (mapMembers.containsKey(member) ) {
+                                    mapMembers.put(member, 
Long.valueOf(System.currentTimeMillis()));
+                                }
+                            }
                         } else {
                             if (log.isInfoEnabled())
-                                log.info("Member[" + resp[i].getSource() + "] 
is not available yet.");
+                                log.info("Member[" + member + "] is not 
available yet.");
                         }
                     } catch (IOException x) {
                         log.error("Unable to deserialize MapMessage.", x);
@@ -543,6 +552,7 @@ public abstract class AbstractReplicated
         } catch (ClassNotFoundException x) {
             log.error("Unable to transfer AbstractReplicatedMap state.", x);
         }
+        this.state = State.STATETRANSFERRED;
     }
 
     /**
@@ -1669,6 +1679,7 @@ public abstract class AbstractReplicated
 
     private static enum State {
         NEW(false),
+        STATETRANSFERRED(false),
         INITIALIZED(true),
         DESTROYED(false);
 

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1805856&r1=1805855&r2=1805856&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Wed Aug 23 06:24:21 2017
@@ -84,6 +84,10 @@
         Add member info to the log message when the failure detection check
         fails in <code>TcpFailureDetector</code>. (kfujino)
       </fix>
+      <fix>
+        Avoid Ping timeout until the added map member by receiving
+        <code>MSG_START</code> message is completely started. (kfujino)
+      </fix>
     </changelog>
   </subsection>
 </section>



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

Reply via email to