Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

read central directory from a stream instead of multiple small random access reads #96

Open
thejoshwolfe opened this issue Nov 12, 2018 · 0 comments · May be fixed by #97
Open

read central directory from a stream instead of multiple small random access reads #96

thejoshwolfe opened this issue Nov 12, 2018 · 0 comments · May be fixed by #97

Comments

@thejoshwolfe
Copy link
Owner

thejoshwolfe commented Nov 12, 2018

The central directory is read from start to finish in a predictable linear way. This perfectly fits the pattern of a read stream, except that yauzl is interested in fixed-size chunks of it at a time. There should be some kind of buffer to put in from of a read stream to provide yauzl with the chunks it wants.

This has promising possibilities for optimizing fs access by reducing the number of reads.

This idea interacts with #92.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant