[hornetq-commits] JBoss hornetq SVN: r9938 - in branches/2_2_0_HA_Improvements_preMerge: src/main/org/hornetq/core/client/impl and 9 other directories.

do-not-reply at jboss.org do-not-reply at jboss.org
Mon Nov 29 12:04:18 EST 2010


Author: ataylor
Date: 2010-11-29 12:04:17 -0500 (Mon, 29 Nov 2010)
New Revision: 9938

Modified:
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/api/core/client/ClusterTopologyListener.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorImpl.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorInternal.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/Topology.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/CoreProtocolManager.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/ClusterTopologyChangeMessage.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/NodeAnnounceMessage.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/ClusterManager.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java
   branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/impl/HornetQServerImpl.java
   branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/FailoverTestBase.java
   branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/MultipleBackupsFailoverTestBase.java
   branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/topology/TopologyClusterTestBase.java
   branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/util/RemoteProcessHornetQServer.java
   branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/jms/server/management/JMSUtil.java
Log:
removed checking for source node when announcing backup as doesnt make sense + test fixes

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/api/core/client/ClusterTopologyListener.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/api/core/client/ClusterTopologyListener.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/api/core/client/ClusterTopologyListener.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -26,7 +26,7 @@
  */
 public interface ClusterTopologyListener
 {
-   void nodeUP(String nodeID, String sourceNodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance);
+   void nodeUP(String nodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance);
    
    void nodeDown(String nodeID);
 }

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -1109,7 +1109,6 @@
             {
                TransportConfiguration config = serverLocator.getClusterTransportConfiguration();
                channel0.send(new NodeAnnounceMessage(serverLocator.getNodeID(),
-                                                     serverLocator.getNodeID(),
                                                      serverLocator.isBackup(),
                                                      config));
             }
@@ -1244,7 +1243,6 @@
             else
             {
                serverLocator.notifyNodeUp(topMessage.getNodeID(),
-                                          topMessage.getSourceNodeID(),
                                           topMessage.getPair(),
                                           topMessage.isLast(),
                                           topMessage.getDistance());

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorImpl.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorImpl.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorImpl.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -1128,7 +1128,6 @@
    }
 
    public synchronized void notifyNodeUp(final String nodeID,
-                                         final String sourceNodeID,
                                          final Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                                          final boolean last,
                                          final int distance)
@@ -1162,7 +1161,7 @@
 
       for (ClusterTopologyListener listener : topologyListeners)
       {
-         listener.nodeUP(nodeID, sourceNodeID, connectorPair, last, distance);
+         listener.nodeUP(nodeID, connectorPair, last, distance);
       }
 
       // Notify if waiting on getting topology

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorInternal.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorInternal.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/ServerLocatorInternal.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -38,7 +38,7 @@
 
    ClientSessionFactory connect() throws  Exception;
 
-   void notifyNodeUp(String nodeID, String sourceNodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance);
+   void notifyNodeUp(String nodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance);
 
    void notifyNodeDown(String nodeID);
 

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/Topology.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/Topology.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/client/impl/Topology.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -39,10 +39,16 @@
     */
    private Map<String, TopologyMember> topology = new HashMap<String, TopologyMember>();
 
+   private boolean debug;
+
    public synchronized boolean addMember(String nodeId, TopologyMember member)
    {
       boolean replaced = false;
       TopologyMember currentMember = topology.get(nodeId);
+      if (debug)
+      {
+         //System.out.println("member.getConnector() = " + member.getConnector());
+      }
       if(currentMember == null)
       {
          topology.put(nodeId, member);
@@ -60,6 +66,15 @@
             currentMember.getConnector().b =  member.getConnector().b;
             replaced = true;
          }
+
+         if(member.getConnector().a == null)
+         {
+            member.getConnector().a = currentMember.getConnector().a;
+         }
+         if(member.getConnector().b == null)
+         {
+            member.getConnector().b = currentMember.getConnector().b;
+         }
       }
       return replaced;
    }
@@ -70,12 +85,12 @@
       return (member != null);
    }
 
-   public synchronized void fireListeners(ClusterTopologyListener listener, String sourceNodeId)
+   public synchronized void fireListeners(ClusterTopologyListener listener)
    {
       int count = 0;
       for (Map.Entry<String, TopologyMember> entry : topology.entrySet())
       {
-         listener.nodeUP(entry.getKey(), sourceNodeId, entry.getValue().getConnector(), ++count == topology.size(), entry.getValue().getDistance());
+         listener.nodeUP(entry.getKey(), entry.getValue().getConnector(), ++count == topology.size(), entry.getValue().getDistance());
       }
    }
 
@@ -149,4 +164,9 @@
       }
       return null;
    }
+
+   public void setDebug(boolean b)
+   {
+      debug = b;
+   }
 }

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/CoreProtocolManager.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/CoreProtocolManager.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/CoreProtocolManager.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -111,16 +111,9 @@
                
                final ClusterTopologyListener listener = new ClusterTopologyListener()
                {
-                  public void nodeUP(String nodeID, String sourceNodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance)
+                  public void nodeUP(String nodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance)
                   {
-                     if(System.getProperty("foo") != null)
-                     {
-                        if(connectorPair.toString().contains("b=org-hornetq-core-remoting-impl-invm-InVMConnectorFactory?server-id=1"))
-                        {
-                           System.out.println("");
-                        }
-                     }
-                     channel0.send(new ClusterTopologyChangeMessage(nodeID, sourceNodeID, connectorPair, last, distance + 1));
+                     channel0.send(new ClusterTopologyChangeMessage(nodeID, connectorPair, last, distance + 1));
                   }
                   
                   public void nodeDown(String nodeID)
@@ -154,7 +147,7 @@
                {
                   pair = new Pair<TransportConfiguration, TransportConfiguration>(msg.getConnector(), null);
                }
-               server.getClusterManager().notifyNodeUp(msg.getNodeID(), msg.getSourceNodeID(), pair, false, 1);
+               server.getClusterManager().notifyNodeUp(msg.getNodeID(), pair, false, 1);
             }
          }
       });

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/ClusterTopologyChangeMessage.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/ClusterTopologyChangeMessage.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/ClusterTopologyChangeMessage.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -34,8 +34,6 @@
    private boolean exit;
    
    private String nodeID;
-
-   private String sourceNodeID;
    
    private Pair<TransportConfiguration, TransportConfiguration> pair;
    
@@ -47,13 +45,11 @@
 
    // Constructors --------------------------------------------------
 
-   public ClusterTopologyChangeMessage(final String nodeID, String sourceNodeID, final Pair<TransportConfiguration, TransportConfiguration> pair, final boolean last, int distance)
+   public ClusterTopologyChangeMessage(final String nodeID, final Pair<TransportConfiguration, TransportConfiguration> pair, final boolean last, int distance)
    {
       super(PacketImpl.CLUSTER_TOPOLOGY);
 
       this.nodeID = nodeID;
-
-      this.sourceNodeID = sourceNodeID;
       
       this.pair = pair;
       
@@ -84,11 +80,6 @@
    {
       return nodeID;
    }
-   
-   public String getSourceNodeID()
-   {
-      return sourceNodeID;
-   }
 
    public Pair<TransportConfiguration, TransportConfiguration> getPair()
    {
@@ -122,7 +113,6 @@
       buffer.writeString(nodeID);
       if (!exit)
       {
-         buffer.writeString(sourceNodeID);
          if (pair.a != null)
          {
             buffer.writeBoolean(true);
@@ -153,7 +143,6 @@
       nodeID = buffer.readString();
       if (!exit)
       {
-         sourceNodeID = buffer.readString();
          boolean hasLive = buffer.readBoolean();
          TransportConfiguration a;
          if(hasLive)

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/NodeAnnounceMessage.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/NodeAnnounceMessage.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/protocol/core/impl/wireformat/NodeAnnounceMessage.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -36,19 +36,15 @@
    
    private TransportConfiguration connector;
 
-   private String sourceNodeID;
-
    // Static --------------------------------------------------------
 
    // Constructors --------------------------------------------------
 
-   public NodeAnnounceMessage(final String nodeID, final String sourceNodeID, final boolean backup, final TransportConfiguration tc)
+   public NodeAnnounceMessage(final String nodeID, final boolean backup, final TransportConfiguration tc)
    {
       super(PacketImpl.NODE_ANNOUNCE);
 
       this.nodeID = nodeID;
-
-      this.sourceNodeID = sourceNodeID;
       
       this.backup = backup;
       
@@ -76,11 +72,6 @@
       return nodeID;
    }
    
-   public String getSourceNodeID()
-   {
-      return sourceNodeID;
-   }
-   
    public boolean isBackup()
    {
       return backup;
@@ -96,7 +87,6 @@
    public void encodeRest(final HornetQBuffer buffer)
    {
       buffer.writeString(nodeID);
-      buffer.writeString(sourceNodeID);
       buffer.writeBoolean(backup);
       connector.encode(buffer);
    }
@@ -105,7 +95,6 @@
    public void decodeRest(final HornetQBuffer buffer)
    {
       this.nodeID = buffer.readString();
-      this.sourceNodeID = buffer.readString();
       this.backup = buffer.readBoolean();
       connector = new TransportConfiguration();
       connector.decode(buffer);

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/ClusterManager.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/ClusterManager.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/ClusterManager.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -50,7 +50,7 @@
 
    void notifyNodeDown(String nodeID);
 
-   void notifyNodeUp(String nodeID, String sourceNodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean backup, int distance);
+   void notifyNodeUp(String nodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean backup, int distance);
 
    Topology getTopology();
 

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -343,23 +343,23 @@
    }
 
    public synchronized void nodeUP(final String nodeID,
-                                   final String sourceNodeID,
                                    final Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                                    final boolean last,
                                    final int distance)
    {
       // discard notifications about ourselves unless its from our backup
+
       if (nodeID.equals(nodeUUID.toString()))
       {
-         if(sourceNodeID.equals(nodeUUID.toString()) && connectorPair.b != null)
+         if(connectorPair.b != null)
          {
-            server.getClusterManager().notifyNodeUp(nodeID, sourceNodeID, connectorPair, last, distance);
+            server.getClusterManager().notifyNodeUp(nodeID, connectorPair, last, distance);
          }
          return;
       }
 
       // we propagate the node notifications to all cluster topology listeners
-      server.getClusterManager().notifyNodeUp(nodeID, sourceNodeID, connectorPair, last, distance);
+      server.getClusterManager().notifyNodeUp(nodeID, connectorPair, last, distance);
 
       // if the node is more than 1 hop away, we do not create a bridge for direct cluster connection
       if (allowsDirectConnectionsOnly && distance > 1 && !allowableConnections.contains(connectorPair.a))

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -251,26 +251,27 @@
    }
 
    public void notifyNodeUp(String nodeID,
-                            String sourceNodeID,
                                    Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                                    boolean last,
                                    int distance)
    {
       boolean updated = topology.addMember(nodeID, new TopologyMember(connectorPair, distance));
+
       if(!updated)
       {
          return;
       }
+      
       for (ClusterTopologyListener listener : clientListeners)
       {
-         listener.nodeUP(nodeID, sourceNodeID, connectorPair, last, distance);
+         listener.nodeUP(nodeID, connectorPair, last, distance);
       }
 
       if (distance < topology.nodes())
       {
          for (ClusterTopologyListener listener : clusterConnectionListeners)
          {
-            listener.nodeUP(nodeID, sourceNodeID, connectorPair, last, distance);
+            listener.nodeUP(nodeID, connectorPair, last, distance);
          }
       }
    }
@@ -313,7 +314,7 @@
       }
 
       // We now need to send the current topology to the client
-      topology.fireListeners(listener, nodeUUID.toString());
+      topology.fireListeners(listener);
    }
 
    public synchronized void removeClusterTopologyListener(final ClusterTopologyListener listener,
@@ -405,12 +406,12 @@
 
          for (ClusterTopologyListener listener : clientListeners)
          {
-            listener.nodeUP(nodeID, nodeID, member.getConnector(), false, member.getDistance());
+            listener.nodeUP(nodeID, member.getConnector(), false, member.getDistance());
          }
 
          for (ClusterTopologyListener listener : clusterConnectionListeners)
          {
-            listener.nodeUP(nodeID, nodeID, member.getConnector(), false, member.getDistance());
+            listener.nodeUP(nodeID, member.getConnector(), false, member.getDistance());
          }
       }
    }
@@ -478,12 +479,12 @@
 
       for (ClusterTopologyListener listener : clientListeners)
       {
-         listener.nodeUP(nodeID, nodeID, member.getConnector(), false, member.getDistance());
+         listener.nodeUP(nodeID, member.getConnector(), false, member.getDistance());
       }
       
       for (ClusterTopologyListener listener : clusterConnectionListeners)
       {
-         listener.nodeUP(nodeID, nodeID, member.getConnector(), false, member.getDistance());
+         listener.nodeUP(nodeID, member.getConnector(), false, member.getDistance());
       }
 
    }
@@ -835,7 +836,7 @@
             try
             {
                ClientSessionFactory backupSessionFactory = backupServerLocator.connect();
-               backupSessionFactory.getConnection().getChannel(0, -1).send(new NodeAnnounceMessage(nodeUUID.toString(), nodeUUID.toString(), true, connector));
+               backupSessionFactory.getConnection().getChannel(0, -1).send(new NodeAnnounceMessage(nodeUUID.toString(), true, connector));
             }
             catch (Exception e)
             {

Modified: branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/impl/HornetQServerImpl.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/impl/HornetQServerImpl.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/src/main/org/hornetq/core/server/impl/HornetQServerImpl.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -200,6 +200,7 @@
    private final Set<ActivateCallback> activateCallbacks = new HashSet<ActivateCallback>();
 
    private volatile GroupingHandler groupingHandler;
+   
    private NodeManager nodeManager;
 
    // Constructors

Modified: branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/FailoverTestBase.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/FailoverTestBase.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/FailoverTestBase.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -398,7 +398,7 @@
          this.latch = latch;
       }
 
-      public void nodeUP(String nodeID, String sourceNodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance)
+      public void nodeUP(String nodeID, Pair<TransportConfiguration, TransportConfiguration> connectorPair, boolean last, int distance)
       {
          if(connectorPair.a != null && !liveNode.contains(connectorPair.a.getName()))
          {

Modified: branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/MultipleBackupsFailoverTestBase.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/MultipleBackupsFailoverTestBase.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/failover/MultipleBackupsFailoverTestBase.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -211,7 +211,7 @@
          this.latch = latch;
       }
 
-      public void nodeUP(String nodeID, String sourceNodeID,
+      public void nodeUP(String nodeID,
                          Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                          boolean last,
                          int distance)

Modified: branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/topology/TopologyClusterTestBase.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/topology/TopologyClusterTestBase.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/topology/TopologyClusterTestBase.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -201,7 +201,7 @@
 
       locator.addClusterTopologyListener(new ClusterTopologyListener()
       {
-         public void nodeUP(String nodeID, String sourceNodeID,
+         public void nodeUP(String nodeID,
                             Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                             boolean last,
                             int distance)
@@ -262,7 +262,7 @@
 
       locator.addClusterTopologyListener(new ClusterTopologyListener()
       {
-         public void nodeUP(String nodeID, String sourceNodeID,
+         public void nodeUP(String nodeID,
                             Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                             boolean last,
                             int distance)
@@ -335,7 +335,7 @@
 
       locator.addClusterTopologyListener(new ClusterTopologyListener()
       {
-         public void nodeUP(String nodeID, String sourceNodeID,
+         public void nodeUP(String nodeID,
                             Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                             boolean last,
                             int distance)
@@ -417,7 +417,7 @@
 
       locator.addClusterTopologyListener(new ClusterTopologyListener()
       {
-         public void nodeUP(String nodeID, String sourceNodeID,
+         public void nodeUP(String nodeID, 
                             Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                             boolean last,
                             int distance)

Modified: branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/util/RemoteProcessHornetQServer.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/util/RemoteProcessHornetQServer.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/cluster/util/RemoteProcessHornetQServer.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -145,6 +145,8 @@
       Assert.assertTrue(ok);
    }
 
+
+
    // Constants -----------------------------------------------------
 
    // Attributes ----------------------------------------------------

Modified: branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/jms/server/management/JMSUtil.java
===================================================================
--- branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/jms/server/management/JMSUtil.java	2010-11-29 14:15:10 UTC (rev 9937)
+++ branches/2_2_0_HA_Improvements_preMerge/tests/src/org/hornetq/tests/integration/jms/server/management/JMSUtil.java	2010-11-29 17:04:17 UTC (rev 9938)
@@ -303,7 +303,6 @@
       }
 
       public void nodeUP(String nodeID,
-                         String sourceNodeID,
                          Pair<TransportConfiguration, TransportConfiguration> connectorPair,
                          boolean last,
                          int distance)



More information about the hornetq-commits mailing list