hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@apache.org
Subject svn commit: r782149 - in /hadoop/hbase/branches/0.19: CHANGES.txt src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java
Date Fri, 05 Jun 2009 21:46:05 GMT
Author: stack
Date: Fri Jun  5 21:46:04 2009
New Revision: 782149

URL: http://svn.apache.org/viewvc?rev=782149&view=rev
Log:
HBASE-1487 ProcessRegionOpen in line 82, LOG.info() outputs 'Bytes.toString(this.startCode)',
which gives those non-printables

Modified:
    hadoop/hbase/branches/0.19/CHANGES.txt
    hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java
    hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java

Modified: hadoop/hbase/branches/0.19/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/CHANGES.txt?rev=782149&r1=782148&r2=782149&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.19/CHANGES.txt Fri Jun  5 21:46:04 2009
@@ -10,6 +10,8 @@
    HBASE-1143  region count erratic in master UI (kill server hosting root
                or meta and see how count goes awry).... make sure you have
                a bunch of reions in there
+   HBASE-1487  ProcessRegionOpen in line 82, LOG.info() outputs
+               "Bytes.toString(this.startCode)", which gives those non-printables
 
 Release 0.19.3 - May 27th, 2009
   BUG FIXES

Modified: hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java?rev=782149&r1=782148&r2=782149&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java
(original)
+++ hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java
Fri Jun  5 21:46:04 2009
@@ -29,12 +29,16 @@
 import org.apache.hadoop.hbase.io.BatchUpdate;
 import org.apache.hadoop.hbase.util.Bytes;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /** 
  * ProcessRegionOpen is instantiated when a region server reports that it is
  * serving a region. This applies to all meta and user regions except the 
  * root region which is handled specially.
  */
 class ProcessRegionOpen extends ProcessRegionStatusChange {
+  static final Log LOG = LogFactory.getLog(ProcessRegionOpen.class);
   protected final HServerAddress serverAddress;
   protected final byte [] startCode;
 
@@ -79,7 +83,7 @@
     // Register the newly-available Region's location.
     LOG.info("updating row " + regionInfo.getRegionNameAsString() +
         " in region " + Bytes.toString(metaRegionName) + " with " +
-        " with startcode " + Bytes.toString(this.startCode) + " and server " +
+        " with startcode " + Bytes.toLong(this.startCode) + " and server " +
         this.serverAddress);
     BatchUpdate b = new BatchUpdate(regionInfo.getRegionName());
     b.put(COL_SERVER,

Modified: hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java?rev=782149&r1=782148&r2=782149&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java
(original)
+++ hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/master/RegionServerOperation.java
Fri Jun  5 21:46:04 2009
@@ -99,4 +99,4 @@
     return Integer.MAX_VALUE;
   }
   protected abstract boolean process() throws IOException;
-}
\ No newline at end of file
+}



Mime
View raw message