guluo2016 commented on code in PR #7487:
URL: https://github.com/apache/hbase/pull/7487#discussion_r2672343759


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java:
##########
@@ -647,7 +647,7 @@ private void unloadRegions(ServerName server, 
List<ServerName> 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);

Review Comment:
   For me is ok.
   
   There are the same issues in other parts of this file as well. Could you 
update them together in this change?
   For example, the log statement a few lines above has the same issue.
   ```java
   LOG.info("Putting server : " + server.getHostname() + " in 
decommission/draining mode");
           admin.decommissionRegionServers(listOfServer, false);
   ```
   @jcongithub 



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

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to