[GitHub] [accumulo] dlmarion commented on a diff in pull request #2906: convert text to printable string before logging in merge tablets

[GitBox <gi...@apache.org>]

dlmarion commented on code in PR #2906:
URL: https://github.com/apache/accumulo/pull/2906#discussion_r959521932


##########
core/src/main/java/org/apache/accumulo/core/util/Merge.java:
##########
@@ -225,8 +226,9 @@ protected void merge(AccumuloClient client, String table, List<Size> sizes, int
     try {
       Text start = sizes.get(0).extent.prevEndRow();
       Text end = sizes.get(numToMerge - 1).extent.endRow();
-      message("Merging %d tablets from (%s to %s]", numToMerge, start == null ? "-inf" : start,
-          end == null ? "+inf" : end);
+      message("Merging %d tablets from (%s to %s]", numToMerge,
+          start == null ? "-inf" : Key.toPrintableString(start.getBytes(), 0, start.getLength(), end.getLength()),

Review Comment:
   I believe the last parameter should be `start.getLength()`.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org