You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2014/12/29 16:45:13 UTC
hbase git commit: HBASE-12762 Region with no hfiles will have the
highest locality cost in LocalityCostFunction (cuijianwei)
Repository: hbase
Updated Branches:
refs/heads/master fbdaf62bf -> 8dac7f9e6
HBASE-12762 Region with no hfiles will have the highest locality cost in LocalityCostFunction (cuijianwei)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8dac7f9e
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8dac7f9e
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8dac7f9e
Branch: refs/heads/master
Commit: 8dac7f9e6c5c22e3d11d22f6a38b7fa9905c6ae8
Parents: fbdaf62
Author: stack <st...@apache.org>
Authored: Mon Dec 29 07:44:46 2014 -0800
Committer: stack <st...@apache.org>
Committed: Mon Dec 29 07:44:46 2014 -0800
----------------------------------------------------------------------
.../hadoop/hbase/master/balancer/StochasticLoadBalancer.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/8dac7f9e/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
----------------------------------------------------------------------
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 5119614..e58f855 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
@@ -1026,7 +1026,9 @@ public class StochasticLoadBalancer extends BaseLoadBalancer {
}
if (index < 0) {
- cost += 1;
+ if (regionLocations.length > 0) {
+ cost += 1;
+ }
} else {
cost += (double) index / (double) regionLocations.length;
}