Author: gdusbabek
Date: Tue Apr  6 16:01:17 2010
New Revision: 931197

URL: http://svn.apache.org/viewvc?rev=931197&view=rev
Log:
remove unused JoinMessage.  Patch by Gary Dusbabek, reviewed by Jonthan Ellis. 
CASSANDRA-827

Removed:
    cassandra/trunk/src/java/org/apache/cassandra/gms/JoinMessage.java
Modified:
    
cassandra/trunk/src/java/org/apache/cassandra/gms/GossipDigestAckMessage.java
    cassandra/trunk/src/java/org/apache/cassandra/gms/Gossiper.java
    cassandra/trunk/src/java/org/apache/cassandra/service/StorageService.java

Modified: 
cassandra/trunk/src/java/org/apache/cassandra/gms/GossipDigestAckMessage.java
URL: 
http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/gms/GossipDigestAckMessage.java?rev=931197&r1=931196&r2=931197&view=diff
==============================================================================
--- 
cassandra/trunk/src/java/org/apache/cassandra/gms/GossipDigestAckMessage.java 
(original)
+++ 
cassandra/trunk/src/java/org/apache/cassandra/gms/GossipDigestAckMessage.java 
Tue Apr  6 16:01:17 2010
@@ -55,11 +55,6 @@ class GossipDigestAckMessage
         epStateMap_ = epStateMap;
     }
     
-    void addGossipDigest(InetAddress ep, int generation, int version)
-    {
-        gDigestList_.add( new GossipDigest(ep, generation, version) );
-    }
-    
     List<GossipDigest> getGossipDigestList()
     {
         return gDigestList_;

Modified: cassandra/trunk/src/java/org/apache/cassandra/gms/Gossiper.java
URL: 
http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/gms/Gossiper.java?rev=931197&r1=931196&r2=931197&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/gms/Gossiper.java (original)
+++ cassandra/trunk/src/java/org/apache/cassandra/gms/Gossiper.java Tue Apr  6 
16:01:17 2010
@@ -177,7 +177,7 @@ public class Gossiper implements IFailur
         EndPointState epState = endPointStateMap_.get(endpoint);
         if (epState.isAlive())
         {
-            logger_.info("InetAddress {} is now dead.", endpoint);;
+            logger_.info("InetAddress {} is now dead.", endpoint);
             isAlive(endpoint, epState, false);
         }
     }
@@ -318,7 +318,7 @@ public class Gossiper implements IFailur
         int index = (size == 1) ? 0 : random_.nextInt(size);
         InetAddress to = liveEndPoints.get(index);
         if (logger_.isTraceEnabled())
-            logger_.trace("Sending a GossipDigestSynMessage to {} ...", to);;
+            logger_.trace("Sending a GossipDigestSynMessage to {} ...", to);
         MessagingService.instance.sendOneWay(message, to);
         return seeds_.contains(to);
     }
@@ -474,21 +474,6 @@ public class Gossiper implements IFailur
         return reqdEndPointState;
     }
 
-    /*
-     * This method is called only from the JoinVerbHandler. This happens
-     * when a new node coming up multicasts the JoinMessage. Here we need
-     * to add the endPoint to the list of live endpoints.
-    */
-    synchronized void join(InetAddress from)
-    {
-        if ( !from.equals( localEndPoint_ ) )
-        {
-            /* Mark this endpoint as "live" */
-               liveEndpoints_.add(from);
-            unreachableEndpoints_.remove(from);
-        }
-    }
-
     void notifyFailureDetector(List<GossipDigest> gDigests)
     {
         IFailureDetector fd = FailureDetector.instance;
@@ -566,7 +551,7 @@ public class Gossiper implements IFailur
         if ( !localState.isAlive() )
         {
             isAlive(addr, localState, true);
-            logger_.info("InetAddress {} is now UP", addr);;
+            logger_.info("InetAddress {} is now UP", addr);
         }
     }
 
@@ -574,13 +559,13 @@ public class Gossiper implements IFailur
     {
         if (justRemovedEndPoints_.containsKey(ep))
             return;
-       logger_.info("Node {} is now part of the cluster", ep);;
+       logger_.info("Node {} is now part of the cluster", ep);
         handleMajorStateChange(ep, epState, false);
     }
 
     private void handleGenerationChange(InetAddress ep, EndPointState epState)
     {
-        logger_.info("Node {} has restarted, now UP again", ep);;
+        logger_.info("Node {} has restarted, now UP again", ep);
         handleMajorStateChange(ep, epState, true);
     }
 
@@ -866,39 +851,6 @@ public class Gossiper implements IFailur
         gossipTimer_ = new Timer(false); // makes the Gossiper reentrant.
     }
 
-    public static class JoinVerbHandler implements IVerbHandler
-    {
-        private static Logger logger_ = LoggerFactory.getLogger( 
JoinVerbHandler.class);
-
-        public void doVerb(Message message)
-        {
-            InetAddress from = message.getFrom();
-            if (logger_.isDebugEnabled())
-              logger_.debug("Received a JoinMessage from {}", from);
-
-            byte[] bytes = message.getMessageBody();
-            DataInputStream dis = new DataInputStream( new 
ByteArrayInputStream(bytes) );
-
-            JoinMessage joinMessage;
-            try
-            {
-                joinMessage = JoinMessage.serializer().deserialize(dis);
-            }
-            catch (IOException e)
-            {
-                throw new RuntimeException(e);
-            }
-            if ( joinMessage.clusterId_.equals( 
DatabaseDescriptor.getClusterName() ) )
-            {
-                Gossiper.instance.join(from);
-            }
-            else
-            {
-                logger_.warn("ClusterName mismatch from " + from + " " + 
joinMessage.clusterId_  + "!=" + DatabaseDescriptor.getClusterName());
-            }
-        }
-    }
-
     public static class GossipDigestSynVerbHandler implements IVerbHandler
     {
         private static Logger logger_ = LoggerFactory.getLogger( 
GossipDigestSynVerbHandler.class);

Modified: 
cassandra/trunk/src/java/org/apache/cassandra/service/StorageService.java
URL: 
http://svn.apache.org/viewvc/cassandra/trunk/src/java/org/apache/cassandra/service/StorageService.java?rev=931197&r1=931196&r2=931197&view=diff
==============================================================================
--- cassandra/trunk/src/java/org/apache/cassandra/service/StorageService.java 
(original)
+++ cassandra/trunk/src/java/org/apache/cassandra/service/StorageService.java 
Tue Apr  6 16:01:17 2010
@@ -103,7 +103,6 @@ public class StorageService implements I
         BOOTSTRAP_TOKEN,
         TREE_REQUEST,
         TREE_RESPONSE,
-        JOIN,
         GOSSIP_DIGEST_SYN,
         GOSSIP_DIGEST_ACK,
         GOSSIP_DIGEST_ACK2,
@@ -224,7 +223,6 @@ public class StorageService implements I
         MessagingService.instance.registerVerbHandlers(Verb.TREE_REQUEST, new 
TreeRequestVerbHandler());
         MessagingService.instance.registerVerbHandlers(Verb.TREE_RESPONSE, new 
AntiEntropyService.TreeResponseVerbHandler());
 
-        MessagingService.instance.registerVerbHandlers(Verb.JOIN, new 
Gossiper.JoinVerbHandler());
         MessagingService.instance.registerVerbHandlers(Verb.GOSSIP_DIGEST_SYN, 
new Gossiper.GossipDigestSynVerbHandler());
         MessagingService.instance.registerVerbHandlers(Verb.GOSSIP_DIGEST_ACK, 
new Gossiper.GossipDigestAckVerbHandler());
         
MessagingService.instance.registerVerbHandlers(Verb.GOSSIP_DIGEST_ACK2, new 
Gossiper.GossipDigestAck2VerbHandler());


Reply via email to