hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nspiegelb...@apache.org
Subject svn commit: r1181516 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
Date Tue, 11 Oct 2011 02:15:51 GMT
Author: nspiegelberg
Date: Tue Oct 11 02:15:50 2011
New Revision: 1181516

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

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
#496626: better handling of large sized blooms

Revert Plan:

Differential Revision: 232597


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
+++ 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;

View raw message