commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Commented] (COMPRESS-234) Patch: TAR InputStream Huge Speed Improvements
Date Sat, 20 Jul 2013 15:54:48 GMT


Sebb commented on COMPRESS-234:

Unfortunately it looks like this breaks the API - a protected field of a public class has
been dropped. However the field is TarBuffer so maybe that could not have been used by 3rd
party code. 

Also please don't introduce protected fields - they make code changes and thorough testing
much harder. All fields should be private - or possibly package - and final where possible.
Only exceptions are final constants.

Also, there are some spurious changes; some of the block start - { - markers have been moved
to the next line, and there are some whitsepace changes. Please only change the minimum needed,
and additional changes make it much harder to review.

[Code layout changes should be done as a separate commit]

> Patch: TAR InputStream Huge Speed Improvements
> ----------------------------------------------
>                 Key: COMPRESS-234
>                 URL:
>             Project: Commons Compress
>          Issue Type: Improvement
>          Components: Archivers
>            Reporter: BELUGA BEHR
>         Attachments: Archiver_Tar.patch,,
> I have looked over TarBuffer And TarArchiveInputStream and found some ways to improve
performance orders of magnitude.
> I used a 1 GB TAR archive file (no compression).
> Times for reading all entry file names:
> Current - 630ms
> Mine - 17ms
> Times for extracting all entry files:
> Current 2446ms
> Mine - 2214ms
> As you can see, I have enhanced the "skip" methods greatly.  Actual extraction was within
a margin of error and the timings bounces around a lot.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message