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:
[email protected]
