This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch branch-2.3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.3 by this push:
     new c146058  HBASE-25306 The log in 
SimpleLoadBalancer#onConfigurationChange is wrong
c146058 is described below

commit c146058e3ae4491ad221ee377ad33b592a67743e
Author: Baiqiang Zhao <zbq.d...@gmail.com>
AuthorDate: Thu Nov 19 22:16:59 2020 +0530

    HBASE-25306 The log in SimpleLoadBalancer#onConfigurationChange is wrong
    
    Closes #2679
    
    Signed-off-by: Viraj Jasani <vjas...@apache.org>
---
 .../org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java
index d8b4256..4435813 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/SimpleLoadBalancer.java
@@ -134,9 +134,9 @@ public class SimpleLoadBalancer extends BaseLoadBalancer {
     float originSlop = slop;
     float originOverallSlop = overallSlop;
     super.setConf(conf);
-    LOG.info("Update configuration of SimpleLoadBalancer, previous slop is "
-            + originSlop + ", current slop is " + slop + "previous overallSlop 
is" +
-            originOverallSlop + ", current overallSlop is " + 
originOverallSlop);
+    LOG.info("Update configuration of SimpleLoadBalancer, previous slop is {},"
+      + " current slop is {}, previous overallSlop is {}, current overallSlop 
is {}",
+      originSlop, slop, originOverallSlop, overallSlop);
   }
 
   private void setLoad(List<ServerAndLoad> slList, int i, int loadChange){

Reply via email to