Skip to content
This repository has been archived by the owner on Jan 11, 2021. It is now read-only.

Add LZ4 compression #98

Merged
merged 1 commit into from
Apr 25, 2018
Merged

Add LZ4 compression #98

merged 1 commit into from
Apr 25, 2018

Conversation

sunchao
Copy link
Owner

@sunchao sunchao commented Apr 22, 2018

This adds support for LZ4 compression codec.

Closes #53 .

@sunchao
Copy link
Owner Author

sunchao commented Apr 22, 2018

This depends on #96 , so should be merged after that.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 94.906% when pulling 70a7c3c on lz4 into 4f99ae1 on master.

@coveralls
Copy link

coveralls commented Apr 22, 2018

Coverage Status

Coverage increased (+0.006%) to 94.96% when pulling 9038898 on lz4 into 8ef823d on master.

Copy link
Collaborator

@sadikovi sadikovi left a comment

Choose a reason for hiding this comment

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

Looks good. Can we have only LZ4Codec part in this PR?

@sadikovi sadikovi mentioned this pull request Apr 23, 2018
@sunchao
Copy link
Owner Author

sunchao commented Apr 24, 2018

Yes will do. I'm not sure how to submit a PR that depends on another PR so it ended up like this..

Copy link
Collaborator

@sadikovi sadikovi left a comment

Choose a reason for hiding this comment

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

Reviewed. Looks good!

@sunchao sunchao merged commit 97806c9 into master Apr 25, 2018
@sunchao
Copy link
Owner Author

sunchao commented Apr 25, 2018

Merged. Thanks @sadikovi !

@sunchao sunchao deleted the lz4 branch April 25, 2018 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants