[COMPRESS-83] GzipCompressorOutputStream should forward write(byte[] buf) and write(byte[] buf, int off, int len) to underlying stream

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0
• Fix Version/s: 1.1

Environment

Linux

• Assignee: Unassigned
• Reporter: Aleksey Shipilev
• Created: 30/Jul/09 13:21
• Updated: 01/Aug/09 20:27
• Resolved: 01/Aug/09 20:18

Description

Current version of GzipCompressorOutputStream decorates only write(int b) method for underlying java.zip.util.GZipOutputStream.

In common scenario of:

OutputStream os = ...;
GzipCompressorOutputStream gcos = new GzipCompressorOutputStream(os);
[do extensive write(byte[] buf]

this results in very slow unbuffered write. This is particularly evident if the code changes to:

OutputStream os = ...;
GZIPOutputStream gos = new GZIPOutputStream(os);
[do extensive write(byte[] buf]

...which is blazingly faster.

In my environment, packing 100 Mb stream with Commons wrapper takes about 120 seconds, while using java.util.zip directly takes less than 1 second.
This performance hit is way too bad for simple wrapper.