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

Autodelete expired items #709

Closed
legale opened this issue Sep 29, 2019 · 7 comments
Closed

Autodelete expired items #709

legale opened this issue Sep 29, 2019 · 7 comments

Comments

@legale
Copy link

@legale legale commented Sep 29, 2019

It would be good to have an ability to autodelete expired items.

@Geolim4
Copy link
Member

@Geolim4 Geolim4 commented Sep 29, 2019

Hello,

You just can't... Since you don't know who's are the expired items except by grabing 'em all (which is unimaginable).
And some cache backend already does that (Redis, Mongo, Couch, Memcache etc)...

Cheers,
Georges

@Yamakasi
Copy link

@Yamakasi Yamakasi commented Nov 21, 2019

You can simply auto delete expired items, your backend just needs to support it. @Geolim4 you build it in yourself, check the Couchbase plugin, that one supports it as Couchbase supports it.

@Geolim4
Copy link
Member

@Geolim4 Geolim4 commented Nov 21, 2019

No, it repeat: I won't implement a feature not supported by 100% of the supported backends....

@Yamakasi
Copy link

@Yamakasi Yamakasi commented Nov 21, 2019

No, it repeat: I won't implement a feature not supported by 100% of the supported backends....

Why is it already in then ?

@Geolim4
Copy link
Member

@Geolim4 Geolim4 commented Nov 21, 2019

@Yamakasi
Copy link

@Yamakasi Yamakasi commented Nov 21, 2019

It's not... Re-read the docs, especially the API 😁
[…]

If you set expired with ->expiresAfter(INT) for an item the backend does the rest if it knows how to handle expired documents/items.

No re-read for the docs, just know how your backend works and implement the right one.

@Geolim4
Copy link
Member

@Geolim4 Geolim4 commented Nov 21, 2019

@Geolim4 Geolim4 pinned this issue May 6, 2020
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
3 participants
You can’t perform that action at this time.