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

File: TAR archive generation #2241

Merged
merged 4 commits into from
Mar 31, 2020
Merged

File: TAR archive generation #2241

merged 4 commits into from
Mar 31, 2020

Conversation

choffmeister
Copy link
Contributor

This PR provides very similar functionality to Archive.zip(), just for uncompressed tarballs. The biggest benefit compared to using the zip flow is, that this does not rely on any blocking Java IO APIs.

References #2034

Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Looks nice and tidy.

@choffmeister
Copy link
Contributor Author

@ennru Thanks for all your feedback. Should now all be integrated in this PR. A little extra change that I made: I renamed TarArchiveHeader -> TarArchiveEntry and this now provides both, the header bytes as well as the trailing padding bytes. This makes the TarArchiveManager cleaner.

Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@ennru ennru added this to the 2.0.0 milestone Mar 31, 2020
@ennru ennru merged commit 6020e28 into akka:master Mar 31, 2020
@ennru
Copy link
Member

ennru commented Mar 31, 2020

Thank you! Great addition.

@ennru ennru mentioned this pull request Apr 14, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants