Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Decompressor's finished() fix #2

wants to merge 1 commit into


None yet
3 participants

Shmuma commented Jan 20, 2012

Without this, hbase 0.90.4-cdh3u2 produces lots of warnings "org.apache.hadoop.hbase.io.hfile.Compression: Deompressor obtained from CodecPool is already finished()"


c0s commented Jan 21, 2012

Wow, good catch! Thanks for submitting the fix. I will test it and then merge.
Please drop me a line if you have a second ;)

omalley commented Jan 28, 2012

This will fail to mark a finished, 0-byte stream as finished.

My version of the fix is at:


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment