You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 04:11:25 UTC

svn commit: r1181469 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java

Author: nspiegelberg
Date: Tue Oct 11 02:11:24 2011
New Revision: 1181469

URL: http://svn.apache.org/viewvc?rev=1181469&view=rev
Log:
Additional debug info added for invalid regionname errors.

Modified:
    hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java?rev=1181469&r1=1181468&r2=1181469&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java Tue Oct 11 02:11:24 2011
@@ -331,7 +331,10 @@ public class HRegionInfo extends Version
         break;
       }
     }
-    if(offset == -1) throw new IOException("Invalid regionName format");
+    if(offset == -1) {
+      throw new IOException("Invalid regionName format: " +
+                            Bytes.toStringBinary(regionName));
+    }
     byte [] tableName = new byte[offset];
     System.arraycopy(regionName, 0, tableName, 0, offset);
     offset = -1;
@@ -341,7 +344,10 @@ public class HRegionInfo extends Version
         break;
       }
     }
-    if(offset == -1) throw new IOException("Invalid regionName format");
+    if(offset == -1) {
+      throw new IOException("Invalid regionName format: " +
+                            Bytes.toStringBinary(regionName));
+    }
     byte [] startKey = HConstants.EMPTY_BYTE_ARRAY;
     if(offset != tableName.length + 1) {
       startKey = new byte[offset - tableName.length - 1];