You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2009/03/19 04:19:00 UTC

svn commit: r755820 - /hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Author: stack
Date: Thu Mar 19 03:18:58 2009
New Revision: 755820

URL: http://svn.apache.org/viewvc?rev=755820&view=rev
Log:
HBASE-1267 binary keys broken in trunk (again) -- part2

Modified:
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=755820&r1=755819&r2=755820&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java Thu Mar 19 03:18:58 2009
@@ -64,7 +64,6 @@
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.hadoop.hbase.util.FSUtils;
 import org.apache.hadoop.hbase.util.Writables;
-import org.apache.hadoop.io.RawComparator;
 import org.apache.hadoop.io.WritableUtils;
 import org.apache.hadoop.util.Progressable;
 import org.apache.hadoop.util.StringUtils;
@@ -1222,9 +1221,6 @@
       // See HRegionServer#RegionListener for how the expire on HRegionServer
       // invokes a HRegion#abort.
       byte [] row = b.getRow();
-      if (this.regionInfo.isMetaRegion()) {
-        LOG.debug("batchUpdate on row for .META.: " + Bytes.toString(row));
-      }
       // If we did not pass an existing row lock, obtain a new one
       Integer lid = getLock(lockid,row);
       long commitTime = (b.getTimestamp() == LATEST_TIMESTAMP) ?
@@ -1956,10 +1952,7 @@
       this.filter = filter;
       this.scanners = new InternalScanner[stores.length];
       try {
-        this.comparator = regionInfo.isRootRegion()?
-            new HStoreKey.RootStoreKeyComparator(): regionInfo.isMetaRegion()?
-              new HStoreKey.MetaStoreKeyComparator():
-                new HStoreKey.StoreKeyComparator();
+        this.comparator = HStoreKey.getRawComparator(regionInfo);
         
         for (int i = 0; i < stores.length; i++) {
           // Only pass relevant columns to each store