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

Probable memory eating for Memcached storage #20

Open
alpha-and-omega opened this issue May 11, 2018 · 0 comments
Open

Probable memory eating for Memcached storage #20

alpha-and-omega opened this issue May 11, 2018 · 0 comments

Comments

@alpha-and-omega
Copy link

alpha-and-omega commented May 11, 2018

I am talking about Memcached storage implementation (probably other too).
Since you use memcached::set with ttl = 0, old unused buckets will stay in memory forever, and this is not good for websites with bucket-per-ip. For websites with 100000 visitors per day it will eat >100 bytes per record = 10Megabytes, and will grow each day. In a month it will eat ~ 300Megabytes and some vps servers will go down.
I think it should be possible to set ttl to specific reasonable value.

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

No branches or pull requests

1 participant