[hornetq-commits] JBoss hornetq SVN: r10936 - in branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core: remoting/impl/invm and 1 other directories.

do-not-reply at jboss.org do-not-reply at jboss.org
Wed Jul 6 15:19:42 EDT 2011


Author: clebert.suconic at jboss.com
Date: 2011-07-06 15:19:41 -0400 (Wed, 06 Jul 2011)
New Revision: 10936

Modified:
   branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java
   branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/remoting/impl/invm/InVMConnector.java
   branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/BridgeImpl.java
   branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java
   branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java
Log:
Fixing a few tests

Modified: branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java
===================================================================
--- branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java	2011-07-06 15:50:53 UTC (rev 10935)
+++ branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/client/impl/ClientSessionFactoryImpl.java	2011-07-06 19:19:41 UTC (rev 10936)
@@ -945,7 +945,10 @@
 
    private void getConnectionWithRetry(final int reconnectAttempts)
    {
-      log.info("getConnectionWithRetry::" + reconnectAttempts);
+      if (log.isTraceEnabled())
+      {
+         log.trace("getConnectionWithRetry::" + reconnectAttempts + " with retryInterval = " + retryInterval + " multiplier = " + retryIntervalMultiplier, new Exception ("trace"));
+      }
 
       long interval = retryInterval;
 

Modified: branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/remoting/impl/invm/InVMConnector.java
===================================================================
--- branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/remoting/impl/invm/InVMConnector.java	2011-07-06 15:50:53 UTC (rev 10935)
+++ branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/remoting/impl/invm/InVMConnector.java	2011-07-06 19:19:41 UTC (rev 10936)
@@ -128,6 +128,8 @@
       if (InVMConnector.failOnCreateConnection)
       {
          InVMConnector.incFailures();
+         
+         log.debug("Returning null on InVMConnector for tests");
          // For testing only
          return null;
       }

Modified: branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/BridgeImpl.java
===================================================================
--- branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/BridgeImpl.java	2011-07-06 15:50:53 UTC (rev 10935)
+++ branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/BridgeImpl.java	2011-07-06 19:19:41 UTC (rev 10936)
@@ -518,8 +518,9 @@
 
    public final void connectionFailed(final HornetQException me, boolean failedOver)
    {
-      log.warn(this + "::Connection failed with failedOver=" + failedOver + "-" + me, new Exception (me.getMessage()));
       
+      log.warn(this + "::Connection failed with failedOver=" + failedOver + "-" + me, me);
+      
       try
       {
          // csf.cleanup();
@@ -538,12 +539,10 @@
       
       if (me.getCode() == HornetQException.DISCONNECTED)
       {
-         log.warn(this + "::Connection failed with failedOver=" + failedOver + "-" + me, me);
          fail(true);
       }
       else
       {
-         log.warn(this + "::Connection failed with failedOver=" + failedOver + "-" + me, me);
          fail(false);
          scheduleRetryConnect();
       }
@@ -612,7 +611,7 @@
    /* This is called only when the bridge is activated */
    protected void connect()
    {
-      BridgeImpl.log.info("Connecting  " + this + " to its destination [" + nodeUUID.toString() + "], csf=" + this.csf);
+      BridgeImpl.log.debug("Connecting  " + this + " to its destination [" + nodeUUID.toString() + "], csf=" + this.csf);
 
       retryCount++;
 
@@ -763,7 +762,7 @@
       
       if (log.isDebugEnabled())
       {
-         log.debug("Scheduling retry for bridge " + this.name + "in " + milliseconds + " milliseconds");
+         log.debug("Scheduling retry for bridge " + this.name + " in " + milliseconds + " milliseconds");
       }
 
       futureScheduledReconnection = scheduledExecutor.schedule(new FutureConnectRunnable(), milliseconds, TimeUnit.MILLISECONDS);

Modified: branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java
===================================================================
--- branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java	2011-07-06 15:50:53 UTC (rev 10935)
+++ branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterConnectionImpl.java	2011-07-06 19:19:41 UTC (rev 10936)
@@ -627,6 +627,10 @@
       targetLocator.setBlockOnNonDurableSend(!useDuplicateDetection);
       targetLocator.setClusterConnection(true);
       
+      targetLocator.setRetryInterval(retryInterval);
+      targetLocator.setMaxRetryInterval(maxRetryInterval);
+      targetLocator.setRetryIntervalMultiplier(retryIntervalMultiplier);
+      
       targetLocator.setNodeID(serverLocator.getNodeID());
       
       targetLocator.setClusterTransportConfiguration(serverLocator.getClusterTransportConfiguration());

Modified: branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java
===================================================================
--- branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java	2011-07-06 15:50:53 UTC (rev 10935)
+++ branches/Branch_2_2_EAP_cluster_clean2/src/main/org/hornetq/core/server/cluster/impl/ClusterManagerImpl.java	2011-07-06 19:19:41 UTC (rev 10936)
@@ -682,9 +682,9 @@
       // We are going to manually retry on the bridge in case of failure
       serverLocator.setReconnectAttempts(0);
       serverLocator.setInitialConnectAttempts(-1);
-
-      
-      
+      serverLocator.setRetryInterval(config.getRetryInterval());
+      serverLocator.setMaxRetryInterval(config.getMaxRetryInterval());
+      serverLocator.setRetryIntervalMultiplier(config.getRetryIntervalMultiplier());
       serverLocator.setClientFailureCheckPeriod(config.getClientFailureCheckPeriod());
       serverLocator.setBlockOnDurableSend(!config.isUseDuplicateDetection());
       serverLocator.setBlockOnNonDurableSend(!config.isUseDuplicateDetection());



More information about the hornetq-commits mailing list