commit 83f5663a229955b04a55b9b0d6cca71b4d597933 Author: stack Date: Fri Oct 2 13:29:39 2015 -0700 HBASE-14519 Purge TestFavoredNodeAssignmentHelper, a test for an abandoned feature that can hang diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestFavoredNodeAssignmentHelper.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestFavoredNodeAssignmentHelper.java index 4dc7d32..522b072 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestFavoredNodeAssignmentHelper.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestFavoredNodeAssignmentHelper.java @@ -39,6 +39,7 @@ import org.apache.hadoop.hbase.util.Bytes; import org.apache.hadoop.hbase.util.Triple; import org.junit.BeforeClass; import org.junit.Test; +import org.junit.Ignore; import org.junit.experimental.categories.Category; import org.mockito.Mockito; @@ -98,7 +99,7 @@ public class TestFavoredNodeAssignmentHelper { return chosenServers; } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSmallCluster() { // Test the case where we cannot assign favored nodes (because the number // of nodes in the cluster is too less) @@ -110,7 +111,7 @@ public class TestFavoredNodeAssignmentHelper { assertFalse(helper.canPlaceFavoredNodes()); } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testPlacePrimaryRSAsRoundRobin() { // Test the regular case where there are many servers in different racks // Test once for few regions and once for many regions @@ -119,7 +120,7 @@ public class TestFavoredNodeAssignmentHelper { primaryRSPlacement(600, null, 10, 10, 10); } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testRoundRobinAssignmentsWithUnevenSizedRacks() { //In the case of uneven racks, the regions should be distributed //proportionately to the rack sizes @@ -136,7 +137,7 @@ public class TestFavoredNodeAssignmentHelper { primaryRSPlacement(459, null, 7, 9, 8); } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSecondaryAndTertiaryPlacementWithSingleRack() { // Test the case where there is a single rack and we need to choose // Primary/Secondary/Tertiary from a single rack. @@ -160,7 +161,7 @@ public class TestFavoredNodeAssignmentHelper { } } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSecondaryAndTertiaryPlacementWithSingleServer() { // Test the case where we have a single node in the cluster. In this case // the primary can be assigned but the secondary/tertiary would be null @@ -178,7 +179,7 @@ public class TestFavoredNodeAssignmentHelper { assertTrue(secondaryAndTertiaryMap.get(regions.get(0)) == null); } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSecondaryAndTertiaryPlacementWithMultipleRacks() { // Test the case where we have multiple racks and the region servers // belong to multiple racks @@ -207,7 +208,7 @@ public class TestFavoredNodeAssignmentHelper { } } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSecondaryAndTertiaryPlacementWithLessThanTwoServersInRacks() { // Test the case where we have two racks but with less than two servers in each // We will not have enough machines to select secondary/tertiary @@ -228,7 +229,7 @@ public class TestFavoredNodeAssignmentHelper { } } - @Test + @Ignore("Disabled for now until FavoredNodes gets finished as a feature") @Test public void testSecondaryAndTertiaryPlacementWithMoreThanOneServerInPrimaryRack() { // Test the case where there is only one server in one rack and another rack // has more servers. We try to choose secondary/tertiary on different