Browse Source

HDFS-8847. change TestHDFSContractAppend to not override testRenameFileBeingAppended method. Contributed by Zhihai Xu

(cherry picked from commit 90b510405af5ac9d4947c4201065b64994f7ef63)
Zhihai Xu 9 years ago
parent
commit
c6e79178d4

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

@@ -764,6 +764,9 @@ Release 2.8.0 - UNRELEASED
     HDFS-8670. Better to exclude decommissioned nodes for namenode NodeUsage JMX
     (J.Andreina via vinayakumarb)
 
+    HDFS-8847. change TestHDFSContractAppend to not override
+    testRenameFileBeingAppended method. (Zhihai Xu)
+
 Release 2.7.2 - UNRELEASED
 
   INCOMPATIBLE CHANGES

+ 0 - 15
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/contract/hdfs/TestHDFSContractAppend.java

@@ -17,11 +17,9 @@ package org.apache.hadoop.fs.contract.hdfs;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.contract.AbstractContractAppendTest;
 import org.apache.hadoop.fs.contract.AbstractFSContract;
-import org.apache.hadoop.fs.contract.ContractTestUtils;
 import org.junit.AfterClass;
 import org.junit.BeforeClass;
 
-import java.io.FileNotFoundException;
 import java.io.IOException;
 
 public class TestHDFSContractAppend extends AbstractContractAppendTest {
@@ -40,17 +38,4 @@ public class TestHDFSContractAppend extends AbstractContractAppendTest {
   protected AbstractFSContract createContract(Configuration conf) {
     return new HDFSContract(conf);
   }
-
-  @Override
-  public void testRenameFileBeingAppended() throws Throwable {
-    try {
-      super.testRenameFileBeingAppended();
-      fail("Expected a FileNotFoundException");
-    } catch (FileNotFoundException e) {
-      // downgrade
-      ContractTestUtils.downgrade("Renaming an open file" +
-                                  "still creates the old path", e);
-
-    }
-  }
 }