Browse Source

HADOOP-9372. Fix bad timeout annotations on tests. Contributed by Arpit Agarwal.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1453637 13f79535-47bb-0310-9956-ffa450edef68
Suresh Srinivas 12 years ago
parent
commit
6942fd1db5

+ 3 - 0
hadoop-common-project/hadoop-common/CHANGES.txt

@@ -453,6 +453,9 @@ Trunk (Unreleased)
     HADOOP-9373. Merge CHANGES.branch-trunk-win.txt to CHANGES.txt.
     HADOOP-9373. Merge CHANGES.branch-trunk-win.txt to CHANGES.txt.
     (suresh)
     (suresh)
 
 
+    HADOOP-9372. Fix bad timeout annotations on tests.
+    (Arpit Agarwal via suresh)
+
 Release 2.0.4-beta - UNRELEASED
 Release 2.0.4-beta - UNRELEASED
 
 
   INCOMPATIBLE CHANGES
   INCOMPATIBLE CHANGES

+ 0 - 3
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/util/TestWinUtils.java

@@ -152,7 +152,6 @@ public class TestWinUtils {
     assertEquals(expected, output);
     assertEquals(expected, output);
   }
   }
 
 
-  @Test (timeout = 30000)
   private void testChmodInternal(String mode, String expectedPerm)
   private void testChmodInternal(String mode, String expectedPerm)
       throws IOException {
       throws IOException {
     File a = new File(TEST_DIR, "file1");
     File a = new File(TEST_DIR, "file1");
@@ -171,7 +170,6 @@ public class TestWinUtils {
     assertFalse(a.exists());
     assertFalse(a.exists());
   }
   }
 
 
-  @Test (timeout = 30000)
   private void testNewFileChmodInternal(String expectedPerm) throws IOException {
   private void testNewFileChmodInternal(String expectedPerm) throws IOException {
     // Create a new directory
     // Create a new directory
     File dir = new File(TEST_DIR, "dir1");
     File dir = new File(TEST_DIR, "dir1");
@@ -193,7 +191,6 @@ public class TestWinUtils {
     assertFalse(dir.exists());
     assertFalse(dir.exists());
   }
   }
 
 
-  @Test (timeout = 30000)
   private void testChmodInternalR(String mode, String expectedPerm,
   private void testChmodInternalR(String mode, String expectedPerm,
       String expectedPermx) throws IOException {
       String expectedPermx) throws IOException {
     // Setup test folder hierarchy
     // Setup test folder hierarchy

+ 3 - 0
hadoop-mapreduce-project/CHANGES.txt

@@ -177,6 +177,9 @@ Trunk (Unreleased)
     MAPREDUCE-4983. Fixed various platform specific assumptions in various tests,
     MAPREDUCE-4983. Fixed various platform specific assumptions in various tests,
     so that they can pass on Windows too. (Chris Nauroth via vinodkv)
     so that they can pass on Windows too. (Chris Nauroth via vinodkv)
 
 
+    HADOOP-9372. Fix bad timeout annotations on tests.
+    (Arpit Agarwal via suresh)
+
 Release 2.0.4-beta - UNRELEASED
 Release 2.0.4-beta - UNRELEASED
 
 
   INCOMPATIBLE CHANGES
   INCOMPATIBLE CHANGES

+ 2 - 2
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapreduce/v2/TestMRJobs.java

@@ -144,7 +144,7 @@ public class TestMRJobs {
     }
     }
   }
   }
 
 
-  @Test (timeout = 30000)
+  @Test (timeout = 300000)
   public void testSleepJob() throws IOException, InterruptedException,
   public void testSleepJob() throws IOException, InterruptedException,
       ClassNotFoundException { 
       ClassNotFoundException { 
     LOG.info("\n\n\nStarting testSleepJob().");
     LOG.info("\n\n\nStarting testSleepJob().");
@@ -542,7 +542,7 @@ public class TestMRJobs {
           trackingUrl.endsWith(jobId.substring(jobId.lastIndexOf("_")) + "/"));
           trackingUrl.endsWith(jobId.substring(jobId.lastIndexOf("_")) + "/"));
   }
   }
   
   
-  @Test (timeout = 30000)
+  @Test (timeout = 300000)
   public void testDistributedCache() throws Exception {
   public void testDistributedCache() throws Exception {
     // Test with a local (file:///) Job Jar
     // Test with a local (file:///) Job Jar
     Path localJobJarPath = makeJobJarWithLib(TEST_ROOT_DIR.toUri().toString());
     Path localJobJarPath = makeJobJarWithLib(TEST_ROOT_DIR.toUri().toString());