From ef120bed7b9e6fab5611262019e6d03ab5fbd0d5 Mon Sep 17 00:00:00 2001 From: Jason Lowe Date: Wed, 10 May 2017 13:02:31 -0500 Subject: [PATCH] HDFS-11745. Increase HDFS test timeouts from 1 second to 10 seconds. Contributed by Eric Badger (cherry picked from commit fab73748a1ce982904d332e6ddd46169f186ad54) --- .../apache/hadoop/hdfs/nfs/nfs3/TestRpcProgramNfs3.java | 2 +- .../org/apache/hadoop/fs/TestSymlinkHdfsFileContext.java | 2 +- .../org/apache/hadoop/fs/TestSymlinkHdfsFileSystem.java | 4 ++-- .../src/test/java/org/apache/hadoop/hdfs/TestDFSUtil.java | 2 +- .../hdfs/server/namenode/metrics/TestNameNodeMetrics.java | 2 +- .../org/apache/hadoop/hdfs/web/TestWebHdfsTokens.java | 8 ++++---- 6 files changed, 10 insertions(+), 10 deletions(-) diff --git a/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestRpcProgramNfs3.java b/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestRpcProgramNfs3.java index 0fabb300db5..f308763b244 100644 --- a/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestRpcProgramNfs3.java +++ b/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestRpcProgramNfs3.java @@ -749,7 +749,7 @@ public class TestRpcProgramNfs3 { assertEquals("Incorrect COMMIT3Response:", null, response2); } - @Test(timeout=1000) + @Test(timeout=10000) public void testIdempotent() { Object[][] procedures = { { Nfs3Constant.NFSPROC3.NULL, 1 }, diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileContext.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileContext.java index d94a5145a97..2a3e4c36371 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileContext.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileContext.java @@ -34,7 +34,7 @@ public class TestSymlinkHdfsFileContext extends TestSymlinkHdfs { wrapper = new FileContextTestWrapper(fc, "/tmp/TestSymlinkHdfsFileContext"); } - @Test(timeout=1000) + @Test(timeout=10000) /** Test access a symlink using AbstractFileSystem */ public void testAccessLinkFromAbstractFileSystem() throws IOException { Path file = new Path(testBaseDir1(), "file"); diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileSystem.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileSystem.java index bf42e24b01d..fba9c42858d 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileSystem.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestSymlinkHdfsFileSystem.java @@ -35,13 +35,13 @@ public class TestSymlinkHdfsFileSystem extends TestSymlinkHdfs { @Override @Ignore("FileSystem adds missing authority in absolute URIs") - @Test(timeout=1000) + @Test(timeout=10000) public void testCreateWithPartQualPathFails() throws IOException {} @Ignore("FileSystem#create creates parent directories," + " so dangling links to directories are created") @Override - @Test(timeout=1000) + @Test(timeout=10000) public void testCreateFileViaDanglingLinkParent() throws IOException {} // Additional tests for DFS-only methods diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSUtil.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSUtil.java index e80cdb4012a..d27cfb60ec6 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSUtil.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSUtil.java @@ -861,7 +861,7 @@ public class TestDFSUtil { DFSUtil.getSpnegoKeytabKey(conf, defaultKey)); } - @Test(timeout=1000) + @Test(timeout=10000) public void testDurationToString() throws Exception { assertEquals("000:00:00:00.000", DFSUtil.durationToString(0)); assertEquals("001:01:01:01.000", diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java index f8af3709912..c22cd1179dd 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java @@ -163,7 +163,7 @@ public class TestNameNodeMetrics { * Test that capacity metrics are exported and pass * basic sanity tests. */ - @Test (timeout = 1800) + @Test (timeout = 10000) public void testCapacityMetrics() throws Exception { MetricsRecordBuilder rb = getMetrics(NS_METRICS); long capacityTotal = MetricsAsserts.getLongGauge("CapacityTotal", rb); diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHdfsTokens.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHdfsTokens.java index c22d5492d23..fccc03fd238 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHdfsTokens.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHdfsTokens.java @@ -130,7 +130,7 @@ public class TestWebHdfsTokens { verify(fs, never()).setDelegationToken((Token)any(Token.class)); } - @Test(timeout = 1000) + @Test(timeout = 10000) public void testGetOpRequireAuth() { for (HttpOpParam.Op op : GetOpParam.Op.values()) { boolean expect = (op == GetOpParam.Op.GETDELEGATIONTOKEN); @@ -138,7 +138,7 @@ public class TestWebHdfsTokens { } } - @Test(timeout = 1000) + @Test(timeout = 10000) public void testPutOpRequireAuth() { for (HttpOpParam.Op op : PutOpParam.Op.values()) { boolean expect = (op == PutOpParam.Op.RENEWDELEGATIONTOKEN || op == PutOpParam.Op.CANCELDELEGATIONTOKEN); @@ -146,14 +146,14 @@ public class TestWebHdfsTokens { } } - @Test(timeout = 1000) + @Test(timeout = 10000) public void testPostOpRequireAuth() { for (HttpOpParam.Op op : PostOpParam.Op.values()) { assertFalse(op.getRequireAuth()); } } - @Test(timeout = 1000) + @Test(timeout = 10000) public void testDeleteOpRequireAuth() { for (HttpOpParam.Op op : DeleteOpParam.Op.values()) { assertFalse(op.getRequireAuth());