[infinispan-commits] Infinispan SVN: r1721 - trunk/core/src/main/java/org/infinispan/util/concurrent/locks.

[infinispan-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2010-04-23 07:30:48 -0400 (Fri, 23 Apr 2010)
New Revision: 1721

Modified:
   trunk/core/src/main/java/org/infinispan/util/concurrent/locks/LockManagerImpl.java
Log:
Better logging of ownership

Modified: trunk/core/src/main/java/org/infinispan/util/concurrent/locks/LockManagerImpl.java
===================================================================
--- trunk/core/src/main/java/org/infinispan/util/concurrent/locks/LockManagerImpl.java	2010-04-23 07:47:07 UTC (rev 1720)
+++ trunk/core/src/main/java/org/infinispan/util/concurrent/locks/LockManagerImpl.java	2010-04-23 11:30:48 UTC (rev 1721)
@@ -61,6 +61,7 @@
    private InvocationContextContainer invocationContextContainer;
    private static final Log log = LogFactory.getLog(LockManagerImpl.class);
    protected static final boolean trace = log.isTraceEnabled();
+   private static final String ANOTHER_THREAD = "(another thread)";
 
    @Inject
    public void injectDependencies(Configuration configuration, TransactionManager transactionManager, InvocationContextContainer invocationContextContainer) {
@@ -133,8 +134,8 @@
          if (l instanceof OwnableReentrantLock) {
             return ((OwnableReentrantLock) l).getOwner();
          } else {
-            // cannot determine owner.
-            return null;
+            // cannot determine owner, JDK Reentrant locks only provide best-effort guesses.
+            return ANOTHER_THREAD;
          }
       } else return null;
    }