petersomogyi commented on a change in pull request #2588: URL: https://github.com/apache/hbase/pull/2588#discussion_r512216019
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/MergeTableRegionsProcedure.java ########## @@ -135,8 +134,8 @@ private static void checkRegionsToMerge(MasterProcedureEnv env, final RegionInfo throw new MergeRegionException(msg); } if (!force && !ri.isAdjacent(previous) && !ri.isOverlap(previous)) { - String msg = "Unable to merge non-adjacent or non-overlapping regions " + - previous.getShortNameToLog() + ", " + ri.getShortNameToLog() + " when force=false"; + String msg = "Unable to merge non-adjacent or non-overlapping regions '" + + previous.getShortNameToLog() + "', '" + ri.getShortNameToLog() + "' when force=false"; Review comment: This is a clean cherry-pick of your commit to master, the `'` character was part of the original commit. 🙂 It was not pushed to branch-2.2 at that time I found this as a nice improvement. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org