From 228ae9aaa40750cb796bbdfd69ba5646c28cd4e7 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. --- 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 495c42d2d7..7dcf8c13d2 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -494,6 +494,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 7246bf50bb..5d4af91c96 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 @@ -827,7 +827,7 @@ public void testGzipLongOverflow() throws IOException { 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. @@ -894,7 +894,7 @@ public void testGzipCodecWriteJava() throws IOException { 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.