[#HADOOP-4950] CompressorStream and BlockCompressorStream should be public

Hadoop Common

CompressorStream and BlockCompressorStream should be public

Created: 28/Dec/08 12:00 AM Updated: 23/Apr/09 07:18 PM

Return to search

Component/s:

Affects Version/s:

None

Fix Version/s:

0.20.0

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	h4950.patch	2008-12-28 05:26 AM	Owen O'Malley	6 kB

Hadoop Flags:	Reviewed
Resolution Date:	08/Jan/09 09:59 PM

Description

« Hide

To simplify writing codecs, CompressionStream and BlockCompressionStream provide helper base classes. They should be made public so they can be used outside of Hadoop's package structure.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Owen O'Malley added a comment - 28/Dec/08 12:04 AM

The trivial patch just makes the classes public. No functional changes are included and therefore no tests are included.

[ Permlink | « Hide ]

Owen O'Malley added a comment - 28/Dec/08 12:13 AM

Correct version of patch.

[ Permlink | « Hide ]

Owen O'Malley added a comment - 28/Dec/08 05:26 AM

I realized that there were some package private methods and fields that also needed to become protected.

[ Permlink | « Hide ]

Chris Douglas added a comment - 08/Jan/09 07:30 PM

+1 Looks good

[ Permlink | « Hide ]

Owen O'Malley added a comment - 08/Jan/09 09:59 PM

I just committed this.