diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestSnapshotTemporaryDirectory.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestSnapshotTemporaryDirectory.java index a8561d0aecae..b1def754f0b6 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestSnapshotTemporaryDirectory.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestSnapshotTemporaryDirectory.java @@ -100,7 +100,8 @@ public class TestSnapshotTemporaryDirectory { * * @throws Exception on failure */ - @BeforeClass public static void setupCluster() throws Exception { + @BeforeClass + public static void setupCluster() throws Exception { setupConf(UTIL.getConfiguration()); UTIL.startMiniCluster(NUM_RS); admin = UTIL.getHBaseAdmin(); @@ -124,7 +125,8 @@ private static void setupConf(Configuration conf) { conf.set(SnapshotDescriptionUtils.SNAPSHOT_WORKING_DIR, "file://" + new Path(TEMP_DIR, ".tmpDir").toUri()); } - @Before public void setup() throws Exception { + @Before + public void setup() throws Exception { HTableDescriptor htd = new HTableDescriptor(TABLE_NAME); htd.setRegionReplication(getNumReplicas()); UTIL.createTable(htd, new byte[][] { TEST_FAM }, UTIL.getConfiguration()); @@ -134,13 +136,15 @@ protected int getNumReplicas() { return 1; } - @After public void tearDown() throws Exception { + @After + public void tearDown() throws Exception { UTIL.deleteTable(TABLE_NAME); SnapshotTestingUtils.deleteAllSnapshots(UTIL.getHBaseAdmin()); SnapshotTestingUtils.deleteArchiveDirectory(UTIL); } - @AfterClass public static void cleanupTest() { + @AfterClass + public static void cleanupTest() { try { UTIL.shutdownMiniCluster(); FileUtils.deleteDirectory(new File(TEMP_DIR)); @@ -149,7 +153,8 @@ protected int getNumReplicas() { } } - @Test(timeout = 180000) public void testRestoreDisabledSnapshot() + @Test + public void testRestoreDisabledSnapshot() throws IOException, InterruptedException { long tid = System.currentTimeMillis(); TableName tableName = TableName.valueOf("testtb-" + tid); @@ -215,7 +220,8 @@ protected int getNumReplicas() { SnapshotTestingUtils.verifyReplicasCameOnline(tableName, admin, getNumReplicas()); } - @Test(timeout = 180000) public void testRestoreEnabledSnapshot() + @Test + public void testRestoreEnabledSnapshot() throws IOException, InterruptedException { long tid = System.currentTimeMillis(); TableName tableName = TableName.valueOf("testtb-" + tid); @@ -282,7 +288,8 @@ protected int getNumReplicas() { * * @throws Exception if snapshot does not complete successfully */ - @Test(timeout = 300000) public void testOfflineTableSnapshot() throws Exception { + @Test + public void testOfflineTableSnapshot() throws Exception { Admin admin = UTIL.getHBaseAdmin(); // make sure we don't fail on listing snapshots SnapshotTestingUtils.assertNoSnapshots(admin); @@ -336,7 +343,8 @@ protected int getNumReplicas() { * * @throws Exception if snapshot does not complete successfully */ - @Test(timeout = 180000) public void testSnapshotCloneContents() throws Exception { + @Test + public void testSnapshotCloneContents() throws Exception { // make sure we don't fail on listing snapshots SnapshotTestingUtils.assertNoSnapshots(admin); @@ -372,7 +380,8 @@ protected int getNumReplicas() { admin.close(); } - @Test(timeout = 180000) public void testOfflineTableSnapshotWithEmptyRegion() throws Exception { + @Test + public void testOfflineTableSnapshotWithEmptyRegion() throws Exception { // test with an empty table with one region // make sure we don't fail on listing snapshots @@ -417,7 +426,7 @@ protected int getNumReplicas() { } // Ensures that the snapshot is transferred to the proper completed snapshot directory - @Test(timeout = 180000) + @Test public void testEnsureTemporaryDirectoryTransfer() throws Exception { Admin admin = UTIL.getAdmin(); TableName tableName2 = TableName.valueOf("testListTableSnapshots"); diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMetaTableMetrics.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMetaTableMetrics.java index 82ce709bd365..16db86db89dd 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMetaTableMetrics.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMetaTableMetrics.java @@ -232,7 +232,7 @@ public void test() throws IOException, InterruptedException { assertEquals(5L, putWithClientMetricsCount); } - @Test(timeout = 30000) + @Test public void testConcurrentAccess() { try { tablename = Bytes.toBytes("hbase:meta"); diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterHandlerFullWhenTransitRegion.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterHandlerFullWhenTransitRegion.java index e136cdd286ff..e41054e98b4e 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterHandlerFullWhenTransitRegion.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterHandlerFullWhenTransitRegion.java @@ -63,7 +63,7 @@ public static void setUp() throws Exception { UTIL.createTable(TableName.valueOf(TABLENAME), "fa"); } - @Test(timeout = 30000) + @Test public void test() throws Exception { RegionInfo regionInfo = UTIL.getAdmin().getRegions(TableName.valueOf(TABLENAME)).get(0); //See HBASE-21754 diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java index e73ba753759a..74d05482d1ea 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java @@ -309,7 +309,7 @@ public ClusterConnection getConnection() { } } - @Test(timeout = 60000) + @Test public void testMasterInitWithSameClientServerZKQuorum() throws Exception { Configuration conf = new Configuration(TESTUTIL.getConfiguration()); conf.set(HConstants.CLIENT_ZOOKEEPER_QUORUM, HConstants.LOCALHOST); @@ -320,7 +320,7 @@ public void testMasterInitWithSameClientServerZKQuorum() throws Exception { master.join(); } - @Test(timeout = 60000) + @Test public void testMasterInitWithObserverModeClientZKQuorum() throws Exception { Configuration conf = new Configuration(TESTUTIL.getConfiguration()); Assert.assertFalse(Boolean.getBoolean(HConstants.CLIENT_ZOOKEEPER_OBSERVER_MODE)); diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/quotas/TestQuotaState.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/quotas/TestQuotaState.java index 73b253c9b0ff..4c359f0d1bc8 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/quotas/TestQuotaState.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/quotas/TestQuotaState.java @@ -203,7 +203,7 @@ public void testQuotaStateUpdateTableThrottle() { assertNoopLimiter(quotaInfo.getTableLimiter(UNKNOWN_TABLE_NAME)); } - @Test(timeout = 60000) + @Test public void testTableThrottleWithBatch() { final TableName TABLE_A = TableName.valueOf("TableA"); final int TABLE_A_THROTTLE_1 = 3; diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/throttle/TestStoreHotnessProtector.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/throttle/TestStoreHotnessProtector.java index 67ac1c352165..218b41d9c61b 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/throttle/TestStoreHotnessProtector.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/throttle/TestStoreHotnessProtector.java @@ -53,7 +53,7 @@ public class TestStoreHotnessProtector { @ClassRule public static final HBaseClassTestRule CLASS_RULE = HBaseClassTestRule.forClass(TestStoreHotnessProtector.class); - @Test(timeout = 60000) + @Test public void testPreparePutCounter() throws Exception { ExecutorService executorService = Executors.newFixedThreadPool(10); diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestAccessController.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestAccessController.java index 523b82f595fd..31f2adf790e7 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestAccessController.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestAccessController.java @@ -3180,7 +3180,7 @@ public Object run() throws Exception { verifyDenied(action, USER_CREATE, USER_RW, USER_RO, USER_NONE, USER_OWNER, USER_ADMIN); } - @Test(timeout = 180000) + @Test public void testGetUserPermissions() throws Throwable { Connection conn = null; try { @@ -3308,7 +3308,7 @@ public void testGetUserPermissions() throws Throwable { } } - @Test(timeout = 180000) + @Test public void testHasPermission() throws Throwable { Connection conn = null; try {