diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index 83e13b7858..96d029c2b0 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -77,6 +77,8 @@ Trunk (unreleased changes) HDFS-3293. Add toString(), equals(..) and hashCode() to JournalInfo. (Hari Mankude via szetszwo) + HDFS-3197. Incorrect class comments in a few tests. (Andy Isaacson via eli) + OPTIMIZATIONS HDFS-2834. Add a ByteBuffer-based read API to DFSInputStream. diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeDeath.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeDeath.java index fbe98dce5a..25a791772a 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeDeath.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeDeath.java @@ -39,8 +39,7 @@ import org.apache.hadoop.hdfs.server.namenode.NameNode; import org.apache.log4j.Level; /** - * This class tests that a file need not be closed before its - * data can be read by another client. + * This class tests that pipelines survive data node death and recovery. */ public class TestDatanodeDeath extends TestCase { { diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeRegistration.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeRegistration.java index 6bb7b45666..71c898bf51 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeRegistration.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDatanodeRegistration.java @@ -38,8 +38,7 @@ import org.apache.hadoop.util.VersionInfo; import org.junit.Test; /** - * This class tests that a file need not be closed before its - * data can be read by another client. + * This class tests data node registration. */ public class TestDatanodeRegistration { diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileCreationClient.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileCreationClient.java index 3516cf6392..428454c9e2 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileCreationClient.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileCreationClient.java @@ -31,11 +31,13 @@ import org.apache.hadoop.hdfs.server.protocol.InterDatanodeProtocol; import org.apache.hadoop.io.IOUtils; import org.apache.log4j.Level; +import org.junit.Test; +import static org.junit.Assert.assertEquals; + /** - * This class tests that a file need not be closed before its - * data can be read by another client. + * This class tests client lease recovery. */ -public class TestFileCreationClient extends junit.framework.TestCase { +public class TestFileCreationClient { static final String DIR = "/" + TestFileCreationClient.class.getSimpleName() + "/"; { @@ -46,6 +48,7 @@ public class TestFileCreationClient extends junit.framework.TestCase { } /** Test lease recovery Triggered by DFSClient. */ + @Test public void testClientTriggeredLeaseRecovery() throws Exception { final int REPLICATION = 3; Configuration conf = new HdfsConfiguration(); diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestReplaceDatanodeOnFailure.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestReplaceDatanodeOnFailure.java index 9a7504a050..5fee500ab3 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestReplaceDatanodeOnFailure.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestReplaceDatanodeOnFailure.java @@ -38,8 +38,7 @@ import org.junit.Assert; import org.junit.Test; /** - * This class tests that a file need not be closed before its - * data can be read by another client. + * This class tests that data nodes are correctly replaced on failure. */ public class TestReplaceDatanodeOnFailure { static final Log LOG = AppendTestUtil.LOG;