You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by te...@apache.org on 2013/11/10 05:04:24 UTC

svn commit: r1540442 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Author: tedyu
Date: Sun Nov 10 04:04:24 2013
New Revision: 1540442

URL: http://svn.apache.org/r1540442
Log:
HBASE-9809 RegionTooBusyException should provide region name which was too busy


Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=1540442&r1=1540441&r2=1540442&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java Sun Nov 10 04:04:24 2013
@@ -5920,7 +5920,11 @@ public class HRegion implements HeapSize
         busyWaitDuration * Math.min(multiplier, maxBusyWaitMultiplier));
       if (!lock.tryLock(waitTime, TimeUnit.MILLISECONDS)) {
         throw new RegionTooBusyException(
-          "failed to get a lock in " + waitTime + "ms");
+          "failed to get a lock in " + waitTime + " ms. " +
+            "regionName=" + (this.getRegionInfo() == null ? "unknown" :
+            this.getRegionInfo().getRegionNameAsString()) +
+            ", server=" + (this.getRegionServerServices() == null ? "unknown" :
+            this.getRegionServerServices().getServerName()));
       }
     } catch (InterruptedException ie) {
       LOG.info("Interrupted while waiting for a lock");