[jboss-cvs] JBossCache/src/org/jboss/cache/lock ...

[Manik Surtani]

  User: msurtani
  Date: 07/05/23 11:22:04

  Modified:    src/org/jboss/cache/lock  LockMap.java
  Log:
  Performance enhancements, including a new invoke() signature for Interceptor
  
  Revision  Changes    Path
  1.13      +4 -7      JBossCache/src/org/jboss/cache/lock/LockMap.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: LockMap.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/lock/LockMap.java,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -b -r1.12 -r1.13
  --- LockMap.java	23 May 2007 10:28:58 -0000	1.12
  +++ LockMap.java	23 May 2007 15:22:04 -0000	1.13
  @@ -6,16 +6,15 @@
    */
   package org.jboss.cache.lock;
   
  -import org.jboss.cache.util.concurrent.ConcurrentHashSet;
  -
   import java.util.Collections;
   import java.util.Set;
  +import java.util.concurrent.CopyOnWriteArraySet;
   
   /**
    * Provide lock ownership mapping.
    *
    * @author Ben Wang
  - * @version $Id: LockMap.java,v 1.12 2007/05/23 10:28:58 msurtani Exp $
  + * @version $Id: LockMap.java,v 1.13 2007/05/23 15:22:04 msurtani Exp $
    */
   public class LockMap
   {
  @@ -25,10 +24,8 @@
   
      private Object writeOwner_ = null;
   
  -   // a CopyOnWriteArraySet is HUGELY inefficient since MANY LockMaps are created and are frequently read from.
  -   // private final Set readOwnerList_ = new CopyOnWriteArraySet();
  -
  -   private final Set readOwnerList_ = new ConcurrentHashSet();
  +   // a CopyOnWriteArraySet is HUGELY inefficient since MANY LockMaps are created and are frequently read from.  Until we have a better impl ...
  +   private final Set readOwnerList_ = new CopyOnWriteArraySet();
   
      public LockMap()
      {