commit d191c7cb6d37039d2582fda1589348c3d01be0d0
author Edward Capriolo <edlinuxguru@gmail.com> Tue May 03 11:15:26 2016 -0400
committer Edward Capriolo <edlinuxguru@gmail.com> Tue May 03 11:15:26 2016 -0400
tree ee41c03a3f461a11dd30be51b008342a280f1ad5
parent a7f90aae08ae76d1cd68d7a1082b76c918526e15

Cluster name was null at this context

src/main/java/com/google/code/gossip/StartupSettings.java

diff --git a/src/main/java/com/google/code/gossip/StartupSettings.java b/src/main/java/com/google/code/gossip/StartupSettings.java
index 6d9d43c..53ed725 100644
--- a/src/main/java/com/google/code/gossip/StartupSettings.java
+++ b/src/main/java/com/google/code/gossip/StartupSettings.java
@@ -62,8 +62,8 @@
    * @param logLevel
    *          unused
    */
-  public StartupSettings(String id, int port, int logLevel) {
-    this(id, port, new GossipSettings());
+  public StartupSettings(String id, int port, int logLevel, String cluster) {
+    this(id, port, new GossipSettings(), cluster);
   }
 
   /**
@@ -74,10 +74,11 @@
    * @param port
    *          The port to start the service on.
    */
-  public StartupSettings(String id, int port, GossipSettings gossipSettings) {
+  public StartupSettings(String id, int port, GossipSettings gossipSettings, String cluster) {
     this.id = id;
     this.port = port;
     this.gossipSettings = gossipSettings;
+    this.setCluster(cluster);
     gossipMembers = new ArrayList<>();
   }
 
@@ -179,25 +180,17 @@
       }
     }
     
-
-    // Lets parse the String as JSON.
     JSONObject jsonObject = new JSONArray(buffer.toString()).getJSONObject(0);
-
-    // Now get the port number.
     int port = jsonObject.getInt("port");
-
-    // Get the id to be used
     String id = jsonObject.getString("id");
-
-    // Get the gossip_interval from the config file.
     int gossipInterval = jsonObject.getInt("gossip_interval");
-
-    // Get the cleanup_interval from the config file.
     int cleanupInterval = jsonObject.getInt("cleanup_interval");
-
-    // Initiate the settings with the port number.
+    String cluster = jsonObject.getString("cluster");
+    if (cluster == null){
+      throw new IllegalArgumentException("cluster was null. It is required");
+    }
     StartupSettings settings = new StartupSettings(id, port, new GossipSettings(gossipInterval,
-            cleanupInterval));
+            cleanupInterval), cluster);
 
     // Now iterate over the members from the config file and add them to the settings.
     String configMembersDetails = "Config-members [";

src/main/java/com/google/code/gossip/manager/PassiveGossipThread.java

diff --git a/src/main/java/com/google/code/gossip/manager/PassiveGossipThread.java b/src/main/java/com/google/code/gossip/manager/PassiveGossipThread.java
index e8eb034..b05a780 100644
--- a/src/main/java/com/google/code/gossip/manager/PassiveGossipThread.java
+++ b/src/main/java/com/google/code/gossip/manager/PassiveGossipThread.java
@@ -65,6 +65,9 @@
               + gossipManager.getMyself().getPort());
       GossipService.LOGGER.debug("I am " + gossipManager.getMyself());
       cluster = gossipManager.getMyself().getClusterName();
+      if (cluster == null){
+        throw new IllegalArgumentException("cluster was null");
+      }
     } catch (SocketException ex) {
       GossipService.LOGGER.warn(ex);
       throw new RuntimeException(ex);