From eb2da051cb3dcd75016e6b21363940f48119e50e Mon Sep 17 00:00:00 2001 From: jc100102 Date: Thu, 27 Nov 2025 16:31:36 -0500 Subject: [PATCH 1/2] HBASE-29733 Use ServerName instead of hostname in unloadRegions log message --- .../src/main/java/org/apache/hadoop/hbase/util/RegionMover.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java index c1f98edd75ab..d10da6446285 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java @@ -647,7 +647,7 @@ private void unloadRegions(ServerName server, List regionServers, break; } LOG.info("Moving {} regions from {} to {} servers using {} threads .Ack Mode: {}", - regionsToMove.size(), this.hostname, regionServers.size(), this.maxthreads, ack); + regionsToMove.size(), server, regionServers.size(), this.maxthreads, ack); Optional metaRegion = getMetaRegionInfoIfToBeMoved(regionsToMove); if (metaRegion.isPresent()) { From 3b4b0cb8d95efd5672758e74d8ac1f3270e92979 Mon Sep 17 00:00:00 2001 From: jc100102 Date: Fri, 5 Dec 2025 23:43:43 -0500 Subject: [PATCH 2/2] HBASE-29733 Correctly use SLF4J placeholder --- .../src/main/java/org/apache/hadoop/hbase/util/RegionMover.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java index d10da6446285..ddc30faa6916 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java @@ -871,7 +871,7 @@ private void includeExcludeRegionServers(String fileName, List regio if (fileName != null) { List servers = readServersFromFile(fileName); if (servers.isEmpty()) { - LOG.warn("No servers provided in the file: {}." + fileName); + LOG.warn("No servers provided in the file: {}.", fileName); return; } Iterator i = regionServers.iterator();