From 9f17e8fc43c0007157d45ac72fae83c3775c9e49 Mon Sep 17 00:00:00 2001 From: Dhruv Parthasarathy Date: Thu, 1 Aug 2013 21:01:02 -0700 Subject: [PATCH] all working --- .../java/com/metamx/druid/master/CostBalancerStrategy.java | 5 +---- .../com/metamx/druid/master/DruidMasterBalancerTest.java | 5 ++++- 2 files changed, 5 insertions(+), 5 deletions(-) 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() {