| Summary: | [CVS Core] Use full stream server compression | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Paul <paul> |
| Component: | Team | Assignee: | Platform Team Inbox <platform-team-inbox> |
| Status: | RESOLVED WONTFIX | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | Keywords: | helpwanted |
| Version: | 2.1 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | |||
|
Description
Paul
I was not aware that a server could be configured to require a compression level to be set. I suspect the problem is that Eclipse makes use of an older compression setting in CVS which only compresses file contents. The reason for this is that previous attempts to compress the whole stream failed due to shortcomings in the java implementation of gzip streams. We should have another look at this to see if the gzip support in Java has been improved (or if the original investigation was faulty). The original assesment was not faulty and it does not look like the 1.4 API has improved the GZIP support. The problem is that the streams provided by Java are for reading and writing files. Thus, only the close() or finish() is guarenteed to flush any internal buffers (flush() appears to do nothing). Supporting full compression requires a different implemention of GZIP streams which has the proper flush behavior. These would either need to be written from scratch or obtained from somewhere else (if an implementation exists with a suitable license). From the newsgroup: We may be able to use these class to create the required streams. ---------------------- Doesn't CVS uses Zlib? If yes, doesn't Java has built-in Zlib support ? java.util.zip.Inflater "This class provides support for general purpose compression using the popular ZLIB compression library" java.util.zip.Deflater "This class provides support for general purpose compression using the popular ZLIB compression library" ?? Post 3.0 There is currently no plan to address this item. As of now 'LATER' and 'REMIND' resolutions are no longer supported. Please reopen this bug if it is still valid for you. |