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:15:51 UTC

svn commit: r1181516 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java

Author: nspiegelberg
Date: Tue Oct 11 02:15:50 2011
New Revision: 1181516

URL: http://svn.apache.org/viewvc?rev=1181516&view=rev
Log:
Allowing [a-f] in hbase-89 region file names to prepare for UUID-based region naming.

Summary:
A trivial regex change in the region file name pattern. We plan to do a rolling
upgrade to this change first. This should have no side effects, but it will
allow to do a rolling upgrade introducing the new region file naming scheme and
adding cache-on-write and evict-on-close to the hbase-89 branch

Test Plan:
Run unit tests.
Run HBaseTest on a 5-node cluster.

Reviewed By: kannan
Reviewers: kenny, jgray, liyintang, kannan
CC: kenny, mbautin, kannan
Tasks:
#496626: better handling of large sized blooms

Revert Plan:
OK

Differential Revision: 232597

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

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java?rev=1181516&r1=1181515&r2=1181516&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java Tue Oct 11 02:15:50 2011
@@ -162,7 +162,7 @@ public class StoreFile {
    * this files id.  Group 2 the referenced region name, etc.
    */
   private static final Pattern REF_NAME_PARSER =
-    Pattern.compile("^(\\d+)(?:\\.(.+))?$");
+    Pattern.compile("^([0-9a-f]+)(?:\\.(.+))?$");
 
   // StoreFile.Reader
   private volatile Reader reader;