[jboss-remoting-commits] JBoss Remoting SVN: r5112 - remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting.

jboss-remoting-commits at lists.jboss.org jboss-remoting-commits at lists.jboss.org
Fri May 1 00:50:39 EDT 2009


Author: ron.sigal at jboss.com
Date: 2009-05-01 00:50:39 -0400 (Fri, 01 May 2009)
New Revision: 5112

Modified:
   remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/ConnectionValidator.java
Log:
Shuts down LeasePinger before notifying ConnectionListeners about bad connection.

Modified: remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/ConnectionValidator.java
===================================================================
--- remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/ConnectionValidator.java	2009-05-01 04:49:36 UTC (rev 5111)
+++ remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/ConnectionValidator.java	2009-05-01 04:50:39 UTC (rev 5112)
@@ -752,7 +752,7 @@
                {
                   public void run()
                   {
-                     log.debug(this + " calling " + listener + ".handleConnectionException() for " + client);
+                     log.debug(ConnectionValidator.this + " calling " + listener + ".handleConnectionException() for " + client);
                      listener.handleConnectionException(t, client);
                   }
                }.start();
@@ -799,7 +799,7 @@
          {
             log.debug(ConnectionValidator.this + "'s connection is invalid");
 
-            notifyListeners(new Exception("Could not connect to server!"));
+//            notifyListeners(new Exception("Could not connect to server!"));
             
             if (stopLeaseOnFailure)
             {
@@ -823,6 +823,8 @@
                
                cancel();
             }
+            
+            notifyListeners(new Exception("Could not connect to server!"));
          }
       }
    }




More information about the jboss-remoting-commits mailing list