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 2010/10/12 22:18:12 UTC

svn commit: r1021912 - /hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java

Author: stack
Date: Tue Oct 12 20:18:12 2010
New Revision: 1021912

URL: http://svn.apache.org/viewvc?rev=1021912&view=rev
Log:
HBASE-3105 Scan.toString fails to escape binary start/stop rows

Modified:
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java?rev=1021912&r1=1021911&r2=1021912&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/Scan.java Tue Oct 12 20:18:12 2010
@@ -440,9 +440,9 @@ public class Scan implements Writable {
   public String toString() {
     StringBuilder sb = new StringBuilder();
     sb.append("startRow=");
-    sb.append(Bytes.toString(this.startRow));
+    sb.append(Bytes.toStringBinary(this.startRow));
     sb.append(", stopRow=");
-    sb.append(Bytes.toString(this.stopRow));
+    sb.append(Bytes.toStringBinary(this.stopRow));
     sb.append(", maxVersions=");
     sb.append(this.maxVersions);
     sb.append(", batch=");
@@ -468,7 +468,7 @@ public class Scan implements Writable {
         sb.append("{");
       }
       sb.append("(family=");
-      sb.append(Bytes.toString(entry.getKey()));
+      sb.append(Bytes.toStringBinary(entry.getKey()));
       sb.append(", columns=");
       if(entry.getValue() == null) {
         sb.append("ALL");
@@ -481,7 +481,7 @@ public class Scan implements Writable {
           } else {
             moreThanOneB = true;
           }
-          sb.append(Bytes.toString(column));
+          sb.append(Bytes.toStringBinary(column));
         }
         sb.append("}");
       }