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

[SPARK-43102][BUILD] Upgrade commons-compress to 1.23.0 #40751

Closed
wants to merge 6 commits into from

Conversation

LuciferYang
Copy link
Contributor

@LuciferYang LuciferYang commented Apr 12, 2023

What changes were proposed in this pull request?

This pr aims to upgrade Apache commons-compress from 1.22 to 1.23.0

Why are the changes needed?

The new version bring some new features:

  • COMPRESS-614: Use FileTime for time fields in SevenZipArchiveEntry
  • COMPRESS-621: Fix calculation the offset of the first ZIP central directory entry
  • COMPRESS-633: Add encryption support for SevenZ
  • COMPRESS-613: Support for extra time data in Zip archives
  • COMPRESS-621: Add org.apache.commons.compress.archivers.zip.DefaultBackingStoreSupplier to write to a custom folder instead of the default temporary folder.
  • COMPRESS-600: Add capability to configure Deflater strategy in GzipCompressorOutputStream: GzipParameters.setDeflateStrategy(int).

and some bugs fix:

  • COMPRESS-638: The GzipCompressorOutputStream#writeHeader() uses ISO_8859_1 to write the file name and comment.
    If the strings contains non-ISO_8859_1 characters, unknown characters are displayed after decompression.
    Use percent encoding for non ISO_8859_1 characters.
  • COMPRESS-641: Add TarArchiveEntry.getLinkFlag()
  • COMPRESS-642: Integer overflow ArithmeticException in TarArchiveOutputStream
  • COMPRESS-642: org.apache.commons.compress.archivers.zip.ZipFile.finalize() should not write to std err.

all changes as follows:

Does this PR introduce any user-facing change?

No

How was this patch tested?

Pass GitHub Actions

@github-actions github-actions bot added the BUILD label Apr 12, 2023
@LuciferYang LuciferYang marked this pull request as draft April 12, 2023 04:57
@LuciferYang LuciferYang marked this pull request as ready for review April 19, 2023 06:16
@zhengruifeng
Copy link
Contributor

merged to master

@LuciferYang
Copy link
Contributor Author

Thanks @zhengruifeng

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

Successfully merging this pull request may close these issues.

2 participants