From e847faf02fdc95854b8d51655564e3cd28b10cbd Mon Sep 17 00:00:00 2001 From: Nelson Ray Date: Tue, 15 Jan 2013 11:29:46 -0800 Subject: [PATCH] change name of MAX_SEGMENTS_TO_MOVE since it's not a final --- .../java/com/metamx/druid/master/BalancerCostAnalyzer.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/server/src/main/java/com/metamx/druid/master/BalancerCostAnalyzer.java b/server/src/main/java/com/metamx/druid/master/BalancerCostAnalyzer.java index 717e3a333a1..8a6f2c17e28 100644 --- a/server/src/main/java/com/metamx/druid/master/BalancerCostAnalyzer.java +++ b/server/src/main/java/com/metamx/druid/master/BalancerCostAnalyzer.java @@ -48,7 +48,7 @@ public class BalancerCostAnalyzer private static final int THIRTY_DAYS_IN_MILLIS = 30 * DAY_IN_MILLIS; private final Random rand; private final DateTime referenceTimestamp; - private int MAX_SEGMENTS_TO_MOVE; + private int maxSegmentsToMove; private List serverHolderList; private double initialTotalCost; private double normalization; @@ -70,7 +70,7 @@ public class BalancerCostAnalyzer this.initialTotalCost = calculateInitialTotalCost(serverHolderList); this.normalization = calculateNormalization(serverHolderList); this.totalSegments = params.getAvailableSegments().size(); - this.MAX_SEGMENTS_TO_MOVE = params.getMaxSegmentsToMove(); + this.maxSegmentsToMove = params.getMaxSegmentsToMove(); } public double getInitialTotalCost() @@ -194,7 +194,7 @@ public class BalancerCostAnalyzer int counter = 0; - while (segmentHoldersToMove.size() < MAX_SEGMENTS_TO_MOVE && counter < 3 * MAX_SEGMENTS_TO_MOVE) { + while (segmentHoldersToMove.size() < maxSegmentsToMove && counter < 3 * maxSegmentsToMove) { counter++; // We want to sample from each server w.p. numSegmentsOnServer / totalSegments