[jboss-cvs] JBossRemoting/src/tests/org/jboss/test/remoting/lease ...

Tom Elrod tom.elrod at jboss.com
Wed Sep 20 14:54:38 EDT 2006


  User: telrod  
  Date: 06/09/20 14:54:38

  Modified:    src/tests/org/jboss/test/remoting/lease 
                        LeaseTestServer.java
  Log:
  JBREM-596 - fix for leasing problem when have multiple clients using same client invoker.  Basically moved all lease managment for clients into the client invokers themselves.
  
  Revision  Changes    Path
  1.5       +9 -0      JBossRemoting/src/tests/org/jboss/test/remoting/lease/LeaseTestServer.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: LeaseTestServer.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/tests/org/jboss/test/remoting/lease/LeaseTestServer.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -b -r1.4 -r1.5
  --- LeaseTestServer.java	9 May 2006 00:59:39 -0000	1.4
  +++ LeaseTestServer.java	20 Sep 2006 18:54:38 -0000	1.5
  @@ -46,6 +46,7 @@
      private static int port = 5400;
   
      private boolean error = false;
  +   private boolean disconnect = false;
   
      private Connector connector = null;
   
  @@ -109,6 +110,10 @@
            connector.stop();
            connector.destroy();
         }
  +      if(!disconnect)
  +      {
  +         throw new RuntimeException("Never got disconnect notification from any of the clients.");
  +      }
      }
   
      public void handleConnectionException(Throwable throwable, Client client)
  @@ -121,6 +126,10 @@
            // need to indicate error.
            error = true;
         }
  +      else
  +      {
  +         disconnect = true;
  +      }
      }
   
      /**
  
  
  



More information about the jboss-cvs-commits mailing list