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

Redis storage adapter with "tags" support #45

Open
xmav opened this issue Sep 12, 2022 · 3 comments
Open

Redis storage adapter with "tags" support #45

xmav opened this issue Sep 12, 2022 · 3 comments

Comments

@xmav
Copy link

xmav commented Sep 12, 2022

Feature Request

Redis storage adapter with "tags" support

Q A
New Feature yes
RFC yes/no
BC Break yes/no

Summary

Did not find existing library for Redis with implementation of TaggableInterface.
Is there any that could be used ? If not would be nice to have this implemented.
(It looks like there was an attempt to do that in the past: zendframework/zend-cache#84)

@boesing boesing transferred this issue from laminas/laminas-cache Sep 13, 2022
@boesing
Copy link
Member

boesing commented Sep 13, 2022

I've moved this to the redis storage adapter of laminas-cache.
Feel free to get your hands on it, since I do neither have any time nor a need for this kind of feature.

But I'd be happy to review any PR towards this feature.

@spanishigirl
Copy link

I support the addition of taggable interface for the Redis Adapter. It makes no sense that other storage adapters have tags support, while Redis does not - with tags support, Redis becomes significantly more useful if you are building applications where proper cache invalidation is critical.

@Ocramius
Copy link
Member

Ocramius commented Dec 8, 2022

Do send patches, please 😁

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

No branches or pull requests

4 participants