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

Add Redis support #9

Open
softinio opened this issue Jan 6, 2020 · 3 comments
Open

Add Redis support #9

softinio opened this issue Jan 6, 2020 · 3 comments

Comments

@softinio
Copy link

softinio commented Jan 6, 2020

Add Redis support as an alternative to sqlite

This would be a great feature as it will make this solution easier to use in an enterprise production environment as getting a redis instance shared amonst multiple instances of your app is very easy and cost effective to use.

@Jonatha-Varjao
Copy link

This is trully a neat idead, I stumped in this repository by acident, stayed by the elegant problem solved approuch. I'm diving into the code base and maybe in a couple of days a can bring an PR.

karlicoss added a commit that referenced this issue Sep 9, 2023
relevant to
- #9
- #14 (thanks to redis expiry might be very easy to implement)
karlicoss added a commit that referenced this issue Sep 9, 2023
relevant to
- #9
- #14 (thanks to redis expiry might be very easy to implement)
@karlicoss
Copy link
Owner

Gave it an initial try (in an isolated test), seems like writing to redis cache might be a bit slow, but reading can be much faster than sqlite!
Supporting in cachew will require a bit of refactoring, especially to support sqlite at the same time, but looks promising so I might try it soon

@karlicoss
Copy link
Owner

had another go at redis, overall don't think it's worth it, writing is much much slower https://github.com/karlicoss/cachew/tree/redis

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

3 participants