From dadb083e41f4aa7112835f7d4c4aefe720100b1d Mon Sep 17 00:00:00 2001 From: Esteban Gutierrez Date: Thu, 30 Oct 2014 10:45:04 -0700 Subject: [PATCH] TestRegionServerNoMaster#testMultipleOpen is flaky after HBASE-11760 --- .../regionserver/TestRegionServerNoMaster.java | 34 ---------------------- 1 file changed, 34 deletions(-) diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerNoMaster.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerNoMaster.java index 24d9dd4..65aed5b 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerNoMaster.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerNoMaster.java @@ -212,40 +212,6 @@ public class TestRegionServerNoMaster { openRegion(HTU, getRS(), hri); } - /** - * Test that we can send multiple openRegion to the region server. - * This is used when: - * - there is a SocketTimeout: in this case, the master does not know if the region server - * received the request before the timeout. - * - We have a socket error during the operation: same stuff: we don't know - * - a master failover: if we find a znode in thz M_ZK_REGION_OFFLINE, we don't know if - * the region server has received the query or not. Only solution to be efficient: re-ask - * immediately. - */ - @Test(timeout = 60000) - public void testMultipleOpen() throws Exception { - - // We close - closeRegionNoZK(); - checkRegionIsClosed(HTU, getRS(), hri); - - // We're sending multiple requests in a row. The region server must handle this nicely. - for (int i = 0; i < 10; i++) { - AdminProtos.OpenRegionRequest orr = RequestConverter.buildOpenRegionRequest( - getRS().getServerName(), hri, null, null); - AdminProtos.OpenRegionResponse responseOpen = getRS().rpcServices.openRegion(null, orr); - Assert.assertTrue(responseOpen.getOpeningStateCount() == 1); - - AdminProtos.OpenRegionResponse.RegionOpeningState ors = responseOpen.getOpeningState(0); - Assert.assertTrue("request " + i + " failed", - ors.equals(AdminProtos.OpenRegionResponse.RegionOpeningState.OPENED) || - ors.equals(AdminProtos.OpenRegionResponse.RegionOpeningState.ALREADY_OPENED) - ); - } - - checkRegionIsOpened(HTU, getRS(), hri); - } - @Test(timeout = 60000) public void testMultipleCloseFromMaster() throws Exception { for (int i = 0; i < 10; i++) { -- 1.8.4