[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting ...

[Ron Sigal]

  User: rsigal  
  Date: 07/08/17 20:35:08

  Modified:    src/main/org/jboss/remoting  Tag:
                        remoting_2_2_2_experimental Lease.java
  Log:
  JBREM-783:  In notifyClientsLost() test !leaseHolders.isEmpty().
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.11.2.1.4.1.2.2 +2 -2      JBossRemoting/src/main/org/jboss/remoting/Lease.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Lease.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/Lease.java,v
  retrieving revision 1.11.2.1.4.1.2.1
  retrieving revision 1.11.2.1.4.1.2.2
  diff -u -b -r1.11.2.1.4.1.2.1 -r1.11.2.1.4.1.2.2
  --- Lease.java	17 Aug 2007 20:25:49 -0000	1.11.2.1.4.1.2.1
  +++ Lease.java	18 Aug 2007 00:35:08 -0000	1.11.2.1.4.1.2.2
  @@ -73,7 +73,7 @@
            }
            else
            {
  -            log.warn(("unrecognized client holder: " + o));
  +            log.warn((clientSessionId + ": unrecognized client holder: " + o.getClass() + ": " + o));
               this.leaseHolders = new ConcurrentHashMap();
            }
         }
  @@ -221,7 +221,7 @@
      private void notifyClientLost()
      {
         // is not for a particular client (but all clients associated with client invoker), so need to inspect request payload for client
  -      if (leaseHolders != null)
  +      if (leaseHolders != null && !leaseHolders.isEmpty())
         {
            // loop through and notify for all clients
            Collection clientHoldersCol = leaseHolders.values();