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

Data compression #1048

Closed
yatsukhnenko opened this issue Dec 7, 2016 · 7 comments
Closed

Data compression #1048

yatsukhnenko opened this issue Dec 7, 2016 · 7 comments
Assignees
Labels
Milestone

Comments

@yatsukhnenko
Copy link
Member

@yatsukhnenko yatsukhnenko commented Dec 7, 2016

It would be useful do data compression before send it to the redis server. This will allow reduce memory usage on the server and network traffic/latency between client and server.

@yatsukhnenko yatsukhnenko added this to the 3.2.0 milestone Dec 7, 2016
@yatsukhnenko yatsukhnenko self-assigned this Dec 7, 2016
@roperto
Copy link

@roperto roperto commented Mar 28, 2017

Possible duplicate of #500

@yatsukhnenko
Copy link
Member Author

@yatsukhnenko yatsukhnenko commented Mar 28, 2017

Duplicate #500

@yatsukhnenko yatsukhnenko reopened this Apr 26, 2017
yatsukhnenko added a commit that referenced this issue May 3, 2017
[WIP] #1048
Compression/decompression is carried out by gzencode/gzdecode and
it is disabled by default.
yatsukhnenko added a commit that referenced this issue May 24, 2017
Compression/decompression is carried out by gzencode/gzdecode and
it is disabled by default.
yatsukhnenko added a commit that referenced this issue May 29, 2017
Issue #1048
This commit is adding support of data compression.
LZF was choosen because it small and fast and Redis server uses it.
Since [pecl package](https://pecl.php.net/package/lzf) doesn't provide
lzf.h file after installing, LZF library was added as submodule.
Another algorythms may be easely added by analogy with serializers.
TODO: unit-tests for different data types.
@yatsukhnenko
Copy link
Member Author

@yatsukhnenko yatsukhnenko commented May 30, 2017

@michael-grunder, review please

@ahmed-sigmalux
Copy link

@ahmed-sigmalux ahmed-sigmalux commented Sep 25, 2017

Has this commit been tested and added to the latest release?

@yatsukhnenko
Copy link
Member Author

@yatsukhnenko yatsukhnenko commented Sep 25, 2017

@ahmed-sigmalux, no, this is for future release

@yatsukhnenko
Copy link
Member Author

@yatsukhnenko yatsukhnenko commented Sep 25, 2017

@michael-grunder, review please

yatsukhnenko added a commit that referenced this issue Oct 9, 2017
This commit is adding support of data compression.
LZF was choosen because it small and fast and Redis server uses it.
Since [pecl package](https://pecl.php.net/package/lzf) doesn't provide
lzf.h file after installing, LZF library was added as submodule.
Another algorythms may be easely added by analogy with serializers.
TODO: unit-tests for different data types.
yatsukhnenko added a commit that referenced this issue Oct 17, 2017
Issue #1048
Add --with-liblzf configure option to use system liblzf.
Use exponentially growing buffer for lzf_decompress.
Move liblzf files to separeted dir in package.xml.
yatsukhnenko added a commit that referenced this issue Oct 17, 2017
This commit is adding support of data compression.
LZF was choosen because it small and fast and Redis server uses it.
Since [pecl package](https://pecl.php.net/package/lzf) doesn't provide
lzf.h file after installing, LZF library was added as submodule.
Another algorythms may be easely added by analogy with serializers.
TODO: unit-tests for different data types.
yatsukhnenko added a commit that referenced this issue Oct 17, 2017
Add --with-liblzf configure option to use system liblzf.
Use exponentially growing buffer for lzf_decompress.
Move liblzf files to separeted dir in package.xml.
@yatsukhnenko
Copy link
Member Author

@yatsukhnenko yatsukhnenko commented Oct 19, 2017

Merged into develop

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

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.