-
Notifications
You must be signed in to change notification settings - Fork 17
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
Entertain the idea of adding gzcompression #17
Comments
My cache size went down from my max of 800mb to ~300mb after implementing compression. It is also highly recommended to compress your values in large scale redis deployment in the redis docs. |
But is it faster? Any change made to this plugin cannot have any performance overhead at all. Remember the credo: RAM is cheap. CPU cycles are expensive. |
If the cache is constantly garbage collecting because it is full its slower. If nothing else this should be an option that can be enabled behind a definition. |
The speed before all else makes me question why we are using phps serialize function instead of phpredis's serialization. Php phpredis's serialization is performed and executed in complied c code and will be faster than php serialize |
Compression is definitely faster for large keys and helps with the RAM usage across the board. My implementation uses a compression marker with the intention of disabling compression somehow for small values. |
Id argue that if you are using a modern version of php such as php fpm or php on windows the cpu use is nothing. |
Drastically needed for each and every claim that is made. |
From #13 by @msigley
8028c9d
We've not seen compression being utilized by any Redis object cache implementation out there so far.
Is it really worth it?
The text was updated successfully, but these errors were encountered: