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.

Sign up for GitHub

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

Jump to bottom

Roadmap to remove LZ4FastDecompressor #178

Open
odaira opened this issue Jun 4, 2021 · 0 comments
Open

Roadmap to remove LZ4FastDecompressor #178

odaira opened this issue Jun 4, 2021 · 0 comments

Comments

@odaira
Copy link
Member

odaira commented Jun 4, 2021 •
edited

Because the original C implementation of LZ4_decompress_fast() has been deprecated since lz4 v1.9.0, lz4-java must sooner or later remove LZ4FastDecompressor, or at least its JNI binding. Here are my thoughts.

  1. Support LZ4SafeDecompressor where only LZ4FastDecompressor is supported
    1. LZ4DecompressorWithLength - e4780ab will be included in lz4-java v1.8.0
    2. LZ4BlockInputStream - lz4-java v1.9.0?
    3. Deprecate the use of LZ4FastDecompressor in LZ4DecompressorWithLength and LZ4BlockInputStream - lz4-java v1.9.0?
  2. Remove the JNI binding of LZ4FastDecompressor; keep the Safe/Unsafe Java versions for the moment - lz4-java v2?
  3. (Remove LZ4FastDecompressor - lz4-java v3?)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@odaira