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

expire check interval #77

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

expire check interval #77

wants to merge 3 commits into from

Conversation

Icemap
Copy link

@Icemap Icemap commented May 6, 2021

I add an expire check interval in gcache.
It can deal the situation when we need delete expire key timely.
Such as using AddedFunc and EvictedFunc to monitoring key's lifecycle.
PTAL.

木村くん、ご覧ください。お願いいたします。
@bluele

@Icemap
Copy link
Author

Icemap commented May 24, 2021

PTAL @bluele

@DoubleDi
Copy link

DoubleDi commented Dec 3, 2021

Hi @bluele!
Sorry for the ping!
Great cache!

Made a try to switch from https://github.com/patrickmn/go-cache as yours has clearly a superior feature - cache size limit.
But, unfortunately, using your LRU based cache I have received multiple memory leaks in my applications.
I am thinking it is because the expired elements are not getting evicted by themselves.

Would be awesome to merge this, as it would be very useful in combined with the expiration feature

dmotylev added a commit to redsift/gcache that referenced this pull request Nov 14, 2022
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.

2 participants