[
https://issues.jboss.org/browse/JBCACHE-1608?page=com.atlassian.jira.plug...
]
Dennis Reed edited comment on JBCACHE-1608 at 11/7/11 4:42 PM:
---------------------------------------------------------------
After JBCACHE-1607 is implemented, this bug will not affect any current code.
So there's no gain by fixing it at this point.
was (Author: dereed):
After JBCACHE-1608 is implemented, this bug will not affect any current code.
IdentityLock#releaseAll() is not recursive
------------------------------------------
Key: JBCACHE-1608
URL:
https://issues.jboss.org/browse/JBCACHE-1608
Project: JBoss Cache
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Locking
Affects Versions: 3.2.8.GA
Reporter: Dennis Reed
Assignee: Dennis Reed
Priority: Minor
IdentityLock#releaseAll(Object owner) is recursive.
IdentityLock#releaseAll() is not recursive.
The only caller of these, NodeBasedLockManager expects these to be the same (recursive)
except for the owner argument.
The result is that NodeBasedLockManager#unlockAll(NodeSPI) does not function correctly
(recursively).
The similar NodeBasedLockManager#unlockAll(NodeSPI, Object owner) does work correctly.
The only caller of NodeBasedLockManager#unlockAll(NodeSPI) is
LegacyRegionManagerImpl#inactivateRegion.
It appears it should be using NodeBasedLockManager#unlockAll(NodeSPI, Object owner)
instead (see JBCACHE-1607).
Since this is the only caller of this method, this bug is considered minor.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see:
http://www.atlassian.com/software/jira