You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by zs...@apache.org on 2009/03/12 06:49:14 UTC
svn commit: r752779 - in /hadoop/core/trunk: CHANGES.txt
src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java
Author: zshao
Date: Thu Mar 12 05:49:13 2009
New Revision: 752779
URL: http://svn.apache.org/viewvc?rev=752779&view=rev
Log:
HADOOP-5379. CBZip2InputStream to throw IOException on data crc error. (Rodrigo Schmidt via zshao)
Modified:
hadoop/core/trunk/CHANGES.txt
hadoop/core/trunk/src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java
Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=752779&r1=752778&r2=752779&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Thu Mar 12 05:49:13 2009
@@ -163,6 +163,9 @@
BUG FIXES
+ HADOOP-5379. CBZip2InputStream to throw IOException on data crc error.
+ (Rodrigo Schmidt via zshao)
+
HADOOP-5326. Fixes CBZip2OutputStream data corruption problem.
(Rodrigo Schmidt via zshao)
Modified: hadoop/core/trunk/src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java?rev=752779&r1=752778&r2=752779&view=diff
==============================================================================
--- hadoop/core/trunk/src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java (original)
+++ hadoop/core/trunk/src/core/org/apache/hadoop/io/compress/bzip2/CBZip2InputStream.java Thu Mar 12 05:49:13 2009
@@ -51,11 +51,9 @@
public class CBZip2InputStream extends InputStream implements BZip2Constants {
private static void reportCRCError() throws IOException {
- // The clean way would be to throw an exception.
- // throw new IOException("crc error");
- // Just print a message, like the previous versions of this class did
- System.err.println("BZip2 CRC error");
+ throw new IOException("BZip2 CRC error");
+
}
private void makeMaps() {