The deprecated HADOOP-6856 constructors can create a compressed writers with a null-wrapped CompressionCodecOption
SequenceFile::Reader discards length for files, does not call openFile
Integrated in Hadoop-Common-trunk #489 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/489/)
Integrated in Hadoop-Common-trunk-Commit #395 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/395/)
This looks great. It passes unit tests.
I just committed this. Thanks, Chris!
Merge with trunk
nod That does look cleaner.
I really like the second iteration of the patch, but maybe we could have compress(kind) and compress(kind, codec) instead of using compress(kind, null) for the default codec. The implementation could be the same.
Collapsed CompressionTypeOption and CompressionCodecOption into a single Option type.