Skip to content

Add support for LZ4 frame format - #142

Open
findepi wants to merge 2 commits into
airlift:masterfrom
findepi:findepi/lz4frame
Open

Add support for LZ4 frame format#142
findepi wants to merge 2 commits into
airlift:masterfrom
findepi:findepi/lz4frame

Conversation

@findepi

@findepi findepi commented Apr 12, 2022

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread pom.xml
<artifactId>lz4-java</artifactId>
<version>1.8.0</version>
<scope>test</scope>
<!-- TODO move to test scope once we have XxHash32 -->

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TODO

Comment on lines +39 to +48
protected boolean isByteBufferSupported()
{
// TODO support byte buffer
return false;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

JVM crashed when testing with direct buffers. I didn't figure out why yet.

Comment thread src/test/java/io/airlift/compress/thirdparty/JPountzLz4FrameCompressor.java Outdated
Comment thread src/main/java/io/airlift/compress/lz4/Lz4FrameDecompressor.java
@findepi

findepi commented Apr 12, 2022

Copy link
Copy Markdown
Contributor Author

( rebased after extracted PRs merged )

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 this pull request may close these issues.

1 participant