You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2014/03/06 21:35:11 UTC
svn commit: r1575031 -
/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
Author: tedyu
Date: Thu Mar 6 20:35:11 2014
New Revision: 1575031
URL: http://svn.apache.org/r1575031
Log:
HBASE-10694 TableSkewCostFunction#cost() casts integral division result to double
Modified:
hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java?rev=1575031&r1=1575030&r2=1575031&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java Thu Mar 6 20:35:11 2014
@@ -732,7 +732,7 @@ public class StochasticLoadBalancer exte
@Override
double cost(Cluster cluster) {
double max = cluster.numRegions;
- double min = cluster.numRegions / cluster.numServers;
+ double min = ((double) cluster.numRegions) / cluster.numServers;
double value = 0;
for (int i = 0; i < cluster.numMaxRegionsPerTable.length; i++) {