Author: kfujino Date: Wed Aug 23 06:15:14 2017 New Revision: 1805853 URL: http://svn.apache.org/viewvc?rev=1805853&view=rev Log: Avoid Ping timeout until the added map member by receiving MSG_START message is completely started.
Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java tomcat/trunk/java/org/apache/catalina/tribes/tipis/LocalStrings.properties tomcat/trunk/webapps/docs/changelog.xml Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java?rev=1805853&r1=1805852&r2=1805853&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java Wed Aug 23 06:15:14 2017 @@ -280,13 +280,23 @@ 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(sm.getString("abstractReplicatedMap.ping.stateTransferredMember", + member)); + if (mapMembers.containsKey(member) ) { + mapMembers.put(member, Long.valueOf(System.currentTimeMillis())); + } + } } else { if (log.isInfoEnabled()) log.info(sm.getString("abstractReplicatedMap.mapMember.unavailable", - resp[i].getSource())); + member)); } } catch (ClassNotFoundException | IOException e) { log.error(sm.getString("abstractReplicatedMap.unable.deserialize.MapMessage"), e); @@ -536,6 +546,7 @@ public abstract class AbstractReplicated } catch (ClassNotFoundException x) { log.error(sm.getString("abstractReplicatedMap.unable.transferState"), x); } + this.state = State.STATETRANSFERRED; } /** @@ -1611,6 +1622,7 @@ public abstract class AbstractReplicated private enum State { NEW(false), + STATETRANSFERRED(false), INITIALIZED(true), DESTROYED(false); Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/LocalStrings.properties URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/tipis/LocalStrings.properties?rev=1805853&r1=1805852&r2=1805853&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/tipis/LocalStrings.properties (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/tipis/LocalStrings.properties Wed Aug 23 06:15:14 2017 @@ -39,6 +39,7 @@ abstractReplicatedMap.leftOver.pingMsg=P abstractReplicatedMap.leftOver.ignored=Message[{0}] is ignored. abstractReplicatedMap.mapMember.unavailable=Member[{0}] is not available yet. abstractReplicatedMap.ping.timeout=Member[{0}] in the Map[{1}] has timed-out in the ping processing. +abstractReplicatedMap.ping.stateTransferredMember=Member[{0}] is state transferred but not available yet. mapMessage.deserialize.error.key=Deserialization error of the MapMessage.key mapMessage.deserialize.error.value=Deserialization error of the MapMessage.value lazyReplicatedMap.unableReplicate.backup=Unable to replicate backup key:[{0}] to backup:[{1}]. Reason:[{2}] Modified: tomcat/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1805853&r1=1805852&r2=1805853&view=diff ============================================================================== --- tomcat/trunk/webapps/docs/changelog.xml (original) +++ tomcat/trunk/webapps/docs/changelog.xml Wed Aug 23 06:15:14 2017 @@ -150,6 +150,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> <subsection name="Other"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org