feat(): add delayed job recipe #41
Conversation
ldlsegovia
force-pushed
the
delayed-job
branch
from
ccddb80 to
e77fbb8
Compare
| @@ -0,0 +1,17 @@ | |||
| if selected?(:delayed_job) | |||
| gather_gem "daemons" | |||
| gather_gem "delayed_job_active_record" | |||
There was a problem hiding this comment.
porque hay que agregar la gema "daemons"??
There was a problem hiding this comment.
There was a problem hiding this comment.
creo que como ahora usamos heroku, y el comando se agrega en el Procfile, ya no hay necesidad de esto.
Heroku se encargara de mantener corriendo ese proceso.
Ademas como bien pusiste en el Procfile ya no usariamos el comando bin/delayed_jobs mencionado ahi, sino que el task que provee activejob...
mmmm, pensandolo bien.... usamos activejob para delayed_jobs en platanus??? cual crees que la practica comun?
There was a problem hiding this comment.
ActiveJob es relativamente nuevo y no he tenido mucha oportunidad de verlo en un proyecto nuevo. Sin embargo, es mucho más simple y escalable usarlo que no (en tanto, si la bd se vuelve un cuello de botella gracias a DJ, uno puede migrar casi con nada a resque, sidekiq u otros). Por eso creo que deberíamos usarlo de ese modo.
There was a problem hiding this comment.
ok,
entonces habria que sacar daemons, ya que vamos a usar activejob, vamos ha echar a andar el proceso con las tareas que provee activejobs, ademas de que heroku se encarga de el proceso.
There was a problem hiding this comment.
De acuerdo con lo que dice @juliogarciag , saco daemons entonces.
ldlsegovia
force-pushed
the
delayed-job
branch
from
6121903 to
169cef6
Compare
|
@blackjid ya saqué |
|
fantastic! |
feat(): add delayed job recipe
No description provided.