[hbase] branch master updated: HBASE-25838 Use double instead of Double in StochasticLoadBalancer (#3221)

[zhangduo]

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

zhangduo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new 762abe3  HBASE-25838 Use double instead of Double in 
StochasticLoadBalancer (#3221)
762abe3 is described below

commit 762abe3beab9dc6711c2497bc3cd7bdb11b95882
Author: Duo Zhang <zhang...@apache.org>
AuthorDate: Tue May 4 09:04:47 2021 +0800

    HBASE-25838 Use double instead of Double in StochasticLoadBalancer (#3221)
    
    Signed-off-by: Yulin Niu <niuyu...@apache.org>
---
 .../hbase/master/balancer/StochasticLoadBalancer.java    | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
index a3e9c58..0ffdc43 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
@@ -139,9 +139,9 @@ public class StochasticLoadBalancer extends 
BaseLoadBalancer {
   private List<CostFunction> costFunctions; // FindBugs: Wants this protected; 
IS2_INCONSISTENT_SYNC
 
   // to save and report costs to JMX
-  private Double curOverallCost = 0d;
-  private Double[] tempFunctionCosts;
-  private Double[] curFunctionCosts;
+  private double curOverallCost = 0d;
+  private double[] tempFunctionCosts;
+  private double[] curFunctionCosts;
 
   // Keep locality based picker and cost function to alert them
   // when new services are offered
@@ -218,8 +218,8 @@ public class StochasticLoadBalancer extends 
BaseLoadBalancer {
     addCostFunction(regionLoadFunctions[4]);
     loadCustomCostFunctions(conf);
 
-    curFunctionCosts= new Double[costFunctions.size()];
-    tempFunctionCosts= new Double[costFunctions.size()];
+    curFunctionCosts= new double[costFunctions.size()];
+    tempFunctionCosts= new double[costFunctions.size()];
 
     isBalancerDecisionRecording = getConf()
       .getBoolean(BaseLoadBalancer.BALANCER_DECISION_BUFFER_ENABLED,
@@ -556,7 +556,7 @@ public class StochasticLoadBalancer extends 
BaseLoadBalancer {
   /**
    * update costs to JMX
    */
-  private void updateStochasticCosts(TableName tableName, Double overall, 
Double[] subCosts) {
+  private void updateStochasticCosts(TableName tableName, double overall, 
double[] subCosts) {
     if (tableName == null) {
       return;
     }
@@ -572,7 +572,7 @@ public class StochasticLoadBalancer extends 
BaseLoadBalancer {
       for (int i = 0; i < costFunctions.size(); i++) {
         CostFunction costFunction = costFunctions.get(i);
         String costFunctionName = costFunction.getClass().getSimpleName();
-        Double costPercent = (overall == 0) ? 0 : (subCosts[i] / overall);
+        double costPercent = (overall == 0) ? 0 : (subCosts[i] / overall);
         // TODO: cost function may need a specific description
         balancer.updateStochasticCost(tableName.getNameAsString(), 
costFunctionName,
           "The percent of " + costFunctionName, costPercent);
@@ -731,7 +731,7 @@ public class StochasticLoadBalancer extends 
BaseLoadBalancer {
       }
 
       Float multiplier = c.getMultiplier();
-      Double cost = c.cost();
+      double cost = c.cost();
 
       this.tempFunctionCosts[i] = multiplier*cost;
       total += this.tempFunctionCosts[i];