From 5ea64e67222d4cb6602b29d7f7c5387a778d616b Mon Sep 17 00:00:00 2001 From: Akira Ajisaka Date: Sat, 4 Apr 2015 00:32:18 +0900 Subject: [PATCH] HADOOP-11800. Clean up some test methods in TestCodec.java. Contributed by Brahma Reddy Battula. (cherry picked from commit 228ae9aaa40750cb796bbdfd69ba5646c28cd4e7) --- hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++ .../test/java/org/apache/hadoop/io/compress/TestCodec.java | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index c0eeb9cba36..261ec545bbd 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -58,6 +58,9 @@ Release 2.8.0 - UNRELEASED HADOOP-11760. Fix typo of javadoc in DistCp. (Brahma Reddy Battula via ozawa). + HADOOP-11800. Clean up some test methods in TestCodec.java. + (Brahma Reddy Battula via aajisaka) + Release 2.7.0 - UNRELEASED INCOMPATIBLE CHANGES diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java index 54768f3459e..ebeb9f5113f 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java @@ -832,7 +832,7 @@ public class TestCodec { br.close(); } - public void testGzipCodecWrite(boolean useNative) throws IOException { + private void testGzipCodecWrite(boolean useNative) throws IOException { // Create a gzipped file using a compressor from the CodecPool, // and try to read it back via the regular GZIPInputStream. @@ -902,7 +902,7 @@ public class TestCodec { public void testGzipNativeCodecWrite() throws IOException { testGzipCodecWrite(true); } - + @Test public void testCodecPoolAndGzipDecompressor() { // BuiltInZlibInflater should not be used as the GzipCodec decompressor. // Assert that this is the case.