-
Notifications
You must be signed in to change notification settings - Fork 70
Add background jobs #353
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 background jobs #353
Conversation
Nice @enricostano! Taking into account that we'll have (and we'll depend on) |
that's a good idea @markets . I wonder though if we can completely get rid of the cache store to simplify the infrastructure as well as the code even further. |
Yes, right now we're using caching (=> and |
Yep, that's why we got it in our radar. 🔪 if possible. |
ebdfbe7
to
d8c72a4
Compare
I whish we could, I remember a conversation with sys admins about redis not being the best option for cache... but I need to dig into that again. It was probably related to performance, I'll look into it. |
Could be. But keep in mind that our use case is rather small. It might be one of those cases where the tradeoff of using it as a cache store makes sense. |
ok @enricostano not really sure (from a perfomance perspective) if redis can compete with memcached at scale. Probably no, since memcached was designed for this purpose. But as @sauloperez said, it's a question of trade-offs, and in our current situation: caching only is used to cache the devise reset passwords, resources are limited (infra, team)... maybe it wouldn't be a bad bad idea. But to be honest, I'd try to revisit #327, think alternatives (remove, reimplement with DB), and probably just 🔥 the cache store for now, as Pau also pointed out:
|
d8c72a4
to
19a1c92
Compare
WAT
We need a background jobs system in order to send emails, mobile push notifications, etc.
Dependencies
coopdevs/timeoverflow-provisioning#47 (Install Redis)
TODO BEFORE DEPLOY
redis
onstaging
(Ansible scripts)redis
by hand inproduction
😬