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:36:46 UTC
svn commit: r1575034 -
/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
Author: tedyu
Date: Thu Mar 6 20:36:46 2014
New Revision: 1575034
URL: http://svn.apache.org/r1575034
Log:
HBASE-10694 TableSkewCostFunction#cost() casts integral division result to double
Modified:
hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
Modified: hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java?rev=1575034&r1=1575033&r2=1575034&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java (original)
+++ hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java Thu Mar 6 20:36:46 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++) {