[jboss-cvs] JBossCache/src/org/jboss/cache/lock ...
Brian Stansberry
brian.stansberry at jboss.com
Mon Jan 29 10:44:35 EST 2007
User: bstansberry
Date: 07/01/29 10:44:35
Modified: src/org/jboss/cache/lock Tag: Branch_JBossCache_1_4_0
IdentityLock.java
Log:
[JBCACHE-951] Reduce log severity when we end up including the log content in an exception message
Revision Changes Path
No revision
No revision
1.12.2.2 +7 -6 JBossCache/src/org/jboss/cache/lock/IdentityLock.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: IdentityLock.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/lock/IdentityLock.java,v
retrieving revision 1.12.2.1
retrieving revision 1.12.2.2
diff -u -b -r1.12.2.1 -r1.12.2.2
--- IdentityLock.java 5 Dec 2006 00:01:51 -0000 1.12.2.1
+++ IdentityLock.java 29 Jan 2007 15:44:35 -0000 1.12.2.2
@@ -56,7 +56,7 @@
*
* @author <a href="mailto:bela at jboss.org">Bela Ban</a> Apr 11, 2003
* @author Ben Wang July 2003
- * @version $Revision: 1.12.2.1 $
+ * @version $Revision: 1.12.2.2 $
*/
public class IdentityLock
{
@@ -175,7 +175,7 @@
wLock = lock_.upgradeLockAttempt(timeout);
} catch (UpgradeException ue) {
String errStr="acquireWriteLock(): lock upgrade failed for " + getFqn() + " (caller=" + caller + ')';
- log.error(errStr, ue);
+ log.debug(errStr, ue);
throw new UpgradeException(errStr, ue);
}
if (wLock == null) {
@@ -183,7 +183,7 @@
map_.removeReader(caller);
String errStr="upgrade lock for " + getFqn() + " could not be acquired after " + timeout + " ms." +
" Lock map ownership " + map_.printInfo() + " (caller=" + caller + ')';
- log.error(errStr);
+ log.debug(errStr);
throw new UpgradeException(errStr);
}
try {
@@ -202,7 +202,7 @@
if (rc == false) {
String errStr = "write lock for " + getFqn() + " could not be acquired after " + timeout + " ms. " +
"Locks: " + map_.printInfo() + " (caller=" + caller + ", lock info: " + toString(true) + ')';
- log.error(errStr);
+ log.debug(errStr);
throw new TimeoutException(errStr);
}
map_.setWriterIfNotNull(caller);
@@ -257,8 +257,9 @@
sb.append("read lock for ").append(getFqn()).append(" could not be acquired by ").append(caller);
sb.append(" after ").append(timeout).append(" ms. " + "Locks: ").append(map_.printInfo());
sb.append(", lock info: ").append(toString(true));
- log.error(sb.toString());
- throw new TimeoutException(sb.toString());
+ String errMsg = sb.toString();
+ log.debug(errMsg);
+ throw new TimeoutException(errMsg);
}
// Only add to the map if we didn't already have the lock
More information about the jboss-cvs-commits
mailing list