diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index d65d7d6ee2f..34ac47a3b4f 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -106,9 +106,6 @@ Release 2.0.5-beta - UNRELEASED HDFS-4609. TestAuditLogs should release log handles between tests. (Ivan Mitic via szetszwo) - HDFS-4584. Skip TestNNWithQJM.testNewNamenodeTakesOverWriter() on Windows. - (Arpit Agarwal via szetszwo) - Release 2.0.4-alpha - UNRELEASED INCOMPATIBLE CHANGES diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/qjournal/TestNNWithQJM.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/qjournal/TestNNWithQJM.java index 9ce2ae3c7ce..a5463d03b39 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/qjournal/TestNNWithQJM.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/qjournal/TestNNWithQJM.java @@ -18,7 +18,6 @@ package org.apache.hadoop.hdfs.qjournal; import static org.junit.Assert.*; -import static org.junit.Assume.*; import java.io.File; import java.io.IOException; @@ -44,7 +43,7 @@ public class TestNNWithQJM { Configuration conf = new HdfsConfiguration(); - private MiniJournalCluster mjc = null; + private MiniJournalCluster mjc; private Path TEST_PATH = new Path("/test-dir"); private Path TEST_PATH_2 = new Path("/test-dir"); @@ -62,11 +61,10 @@ public void startJNs() throws Exception { public void stopJNs() throws Exception { if (mjc != null) { mjc.shutdown(); - mjc = null; } } - @Test (timeout = 30000) + @Test public void testLogAndRestart() throws IOException { conf.set(DFSConfigKeys.DFS_NAMENODE_NAME_DIR_KEY, MiniDFSCluster.getBaseDirectory() + "/TestNNWithQJM/image"); @@ -95,12 +93,9 @@ public void testLogAndRestart() throws IOException { cluster.shutdown(); } } - - @Test (timeout = 30000) + + @Test public void testNewNamenodeTakesOverWriter() throws Exception { - // Skip the test on Windows. See HDFS-4584. - assumeTrue(!Path.WINDOWS); - File nn1Dir = new File( MiniDFSCluster.getBaseDirectory() + "/TestNNWithQJM/image-nn1"); File nn2Dir = new File( @@ -159,7 +154,7 @@ public void testNewNamenodeTakesOverWriter() throws Exception { } } - @Test (timeout = 30000) + @Test public void testMismatchedNNIsRejected() throws Exception { conf.set(DFSConfigKeys.DFS_NAMENODE_NAME_DIR_KEY, MiniDFSCluster.getBaseDirectory() + "/TestNNWithQJM/image"); @@ -193,8 +188,8 @@ public void testMismatchedNNIsRejected() throws Exception { "Unable to start log segment 1: too few journals", ioe); } } - - @Test (timeout = 30000) + + @Test public void testWebPageHasQjmInfo() throws Exception { conf.set(DFSConfigKeys.DFS_NAMENODE_NAME_DIR_KEY, MiniDFSCluster.getBaseDirectory() + "/TestNNWithQJM/image");