diff --git a/server/src/main/java/com/metamx/druid/master/CostBalancerStrategy.java b/server/src/main/java/com/metamx/druid/master/CostBalancerStrategy.java index e7c01bc6dff..ca958d7d2d3 100644 --- a/server/src/main/java/com/metamx/druid/master/CostBalancerStrategy.java +++ b/server/src/main/java/com/metamx/druid/master/CostBalancerStrategy.java @@ -52,10 +52,7 @@ public class CostBalancerStrategy implements BalancerStrategy { return holder; } - else - { - return null; - } + return null; } diff --git a/server/src/test/java/com/metamx/druid/master/DruidMasterBalancerTest.java b/server/src/test/java/com/metamx/druid/master/DruidMasterBalancerTest.java index 87ada744cb9..84bd90f157f 100644 --- a/server/src/test/java/com/metamx/druid/master/DruidMasterBalancerTest.java +++ b/server/src/test/java/com/metamx/druid/master/DruidMasterBalancerTest.java @@ -133,7 +133,7 @@ public class DruidMasterBalancerTest } @Test - public void testMoveToEmptyServer() + public void testMoveToEmptyServerBalancer() { EasyMock.expect(druidServer1.getName()).andReturn("from").atLeastOnce(); EasyMock.expect(druidServer1.getCurrSize()).andReturn(30L).atLeastOnce(); @@ -200,6 +200,9 @@ public class DruidMasterBalancerTest Assert.assertTrue(params.getMasterStats().getPerTierStats().get("movedCount").get("normal").get() < segments.size()); } + + + @Test public void testRun1() {