hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rodrigo Schmidt <rschm...@facebook.com>
Subject Why don't we throw an exception for CRC errors on CBZip2InputStream.java?
Date Mon, 02 Mar 2009 23:46:26 GMT

>From org.apache.hadoop.io.compress.bzip2.CBZip2InputStream.java:

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");

Is there any reason for this behavior?

The CRC code is working fine and CRC verification is useful to identify eventual bugs on CBZip2OutputStream.java.

I'm willing to change the method above if there's no objection to that:



  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message