svn commit: r1181361 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java

[ns...@apache.org]

Author: nspiegelberg
Date: Tue Oct 11 02:01:55 2011
New Revision: 1181361

URL: http://svn.apache.org/viewvc?rev=1181361&view=rev
Log:
HBase FSCK: Correction. Ignore Non-Hex Directories

Summary:
Original patch ignored non-numeric directories.  Directories are actually a raw
hash, so we need to include all hex directories.

Test Plan:
bin/hbase hbck 2>&1 | grep -v healthy

DiffCamp Revision: 149034
Reviewed By: kannan
CC: nspiegelberg, kannan, kranganathan
Revert Plan:
OK

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

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java?rev=1181361&r1=1181360&r2=1181361&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java Tue Oct 11 02:01:55 2011
@@ -190,8 +190,8 @@ public class HBaseFsck {
       for (FileStatus regionDir : regionDirs) {
         String encodedName = regionDir.getPath().getName();
 
-        // ignore directories that aren't numeric
-        if (!encodedName.matches("^\\d+$")) continue;
+        // ignore directories that aren't hexadecimal
+        if (!encodedName.toLowerCase().matches("[0-9a-f]+")) continue;
 
         HbckInfo hbi = getOrCreateInfo(encodedName);
         hbi.foundRegionDir = regionDir;