all working

This commit is contained in:
Dhruv Parthasarathy 2013-08-01 21:01:02 -07:00
parent 696b04020b
commit 9f17e8fc43
2 changed files with 5 additions and 5 deletions

View File

@ -52,10 +52,7 @@ public class CostBalancerStrategy implements BalancerStrategy
{ {
return holder; return holder;
} }
else return null;
{
return null;
}
} }

View File

@ -133,7 +133,7 @@ public class DruidMasterBalancerTest
} }
@Test @Test
public void testMoveToEmptyServer() public void testMoveToEmptyServerBalancer()
{ {
EasyMock.expect(druidServer1.getName()).andReturn("from").atLeastOnce(); EasyMock.expect(druidServer1.getName()).andReturn("from").atLeastOnce();
EasyMock.expect(druidServer1.getCurrSize()).andReturn(30L).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()); Assert.assertTrue(params.getMasterStats().getPerTierStats().get("movedCount").get("normal").get() < segments.size());
} }
@Test @Test
public void testRun1() public void testRun1()
{ {