Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "HBASE-18164 Fast locality computation in balancer - addendum handles NaN" 

This reverts commit 74e9d31474.

early commit missed some review feedback.
This commit is contained in:
Sean Busbey 2017-06-27 13:33:23 -05:00
parent a88310e9ae
commit 386a6e87fe
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/StochasticLoadBalancer.java
View File

@ -1279,9 +1279,8 @@ public class StochasticLoadBalancer extends BaseLoadBalancer {
} }
// We normalize locality to be a score between 0 and 1.0 representing how good it // We normalize locality to be a score between 0 and 1.0 representing how good it
// is compared to how good it could be. If bestLocality is 0, assume locality is 100 // is compared to how good it could be
// (and the cost is 0) locality /= bestLocality;
locality = bestLocality == 0 ? 1 : locality / bestLocality;
} }
@Override @Override
@ -1292,7 +1291,7 @@ public class StochasticLoadBalancer extends BaseLoadBalancer {
return; return;
} }
double localityDelta = getWeightedLocality(region, newEntity) - getWeightedLocality(region, oldEntity); double localityDelta = getWeightedLocality(region, newEntity) - getWeightedLocality(region, oldEntity);
double normalizedDelta = bestLocality == 0 ? 0.0 : localityDelta / bestLocality; double normalizedDelta = localityDelta / bestLocality;
locality += normalizedDelta; locality += normalizedDelta;
} }