svn commit: r777190 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/HServerInfo.java

[st...@apache.org]

Author: stack
Date: Thu May 21 17:29:18 2009
New Revision: 777190

URL: http://svn.apache.org/viewvc?rev=777190&view=rev
Log:
HBASE-1272 Unreadable log messages -- ... to the only server localhost_1237525439599_56094 <- You'd have to be perverse to recognize that as a hostname, startcode, and port

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/HServerInfo.java

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=777190&r1=777189&r2=777190&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Thu May 21 17:29:18 2009
@@ -150,6 +150,9 @@
    HBASE-1257  base64 encoded values are not contained in quotes during the
                HBase REST JSON serialization (Brian Beggs via Stack)
    HBASE-1436  Killing regionserver can make corrupted hfile
+   HBASE-1272  Unreadable log messages -- "... to the only server
+               localhost_1237525439599_56094" <- You'd have to be perverse
+               to recognize that as a hostname, startcode, and port
 
   IMPROVEMENTS
    HBASE-1089  Add count of regions on filesystem to master UI; add percentage

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/HServerInfo.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/HServerInfo.java?rev=777190&r1=777189&r2=777190&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/HServerInfo.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/HServerInfo.java Thu May 21 17:29:18 2009
@@ -245,10 +245,10 @@
 
   private static String getServerName(String hostName, int port, long startCode) {
     StringBuilder name = new StringBuilder(hostName);
-    name.append("_");
-    name.append(startCode);
-    name.append("_");
+    name.append(",");
     name.append(port);
+    name.append(",");
+    name.append(startCode);
     return name.toString();
   }
 }