User: patriot1burke
Date: 01/08/07 16:26:12
Modified: src/main/org/jboss/ejb/plugins AbstractInstanceCache.java
Log:
releaseLockRef should be enclosed in peek in remove()
Revision Changes Path
1.19 +8 -4 jboss/src/main/org/jboss/ejb/plugins/AbstractInstanceCache.java
Index: AbstractInstanceCache.java
===================================================================
RCS file:
/cvsroot/jboss/jboss/src/main/org/jboss/ejb/plugins/AbstractInstanceCache.java,v
retrieving revision 1.18
retrieving revision 1.19
diff -u -r1.18 -r1.19
--- AbstractInstanceCache.java 2001/08/07 23:22:56 1.18
+++ AbstractInstanceCache.java 2001/08/07 23:26:11 1.19
@@ -63,7 +63,7 @@
* @author <a href="[EMAIL PROTECTED]">Bill Burke</a>
* @author <a href="[EMAIL PROTECTED]">Marc Fleury</a>
*
- * @version $Revision: 1.18 $
+ * @version $Revision: 1.19 $
*
* <p><b>Revisions:</b>
*
@@ -82,6 +82,10 @@
* encapsulated in BeanLock and BeanLockManager. Did this because the lifetime
* of an EntityLock is sometimes longer than the lifetime of the ctx.
* </ol>
+ * <p><b>2001/08/07: billb</b>
+ * <ol>
+ * <li>releaseLockRef should be enclosed in peek in remove()
+ * </ol>
*/
public abstract class AbstractInstanceCache
implements InstanceCache, XmlLoadable, Monitorable, MetricsConstants
@@ -280,11 +284,11 @@
if (getCache().peek(id) != null)
{
getCache().remove(id);
+ // When we introduced the bean in the cache we used the get to increase
the lock count
+ // now we decrease it and if the count is zero the manager will release
the object from memory
+ getContainer().getLockManager().removeLockRef(id);
}
}
- // When we introduced the bean in the cache we used the get to increase the
lock count
- // now we decrease it and if the count is zero the manager will release the
object from memory
- getContainer().getLockManager().removeLockRef(id);
}
public boolean isActive(Object id)
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development
