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.

Sign up for GitHub

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

Jump to bottom

ActiveJob #520

Closed
ventsislaf opened this issue Feb 16, 2015 · 2 comments
Closed

ActiveJob #520

ventsislaf opened this issue Feb 16, 2015 · 2 comments

Comments

@ventsislaf
Copy link
Contributor

What are thoughbot's thoughts (how it sounds!) on using ActiveJob? Do you think it could bring some value over using DelayedJob directly?

I was thinking to open a pull request in which ActiveJob's queue adapter is configured to :delayed_job but I want to hear some of your opinions first.
@derekprior
Copy link
Contributor

I think we should be using ActiveJob by default, breaking out to queue-specific code only when the ActiveJob API is insufficient.

@tute
Copy link
Contributor

tute commented Feb 20, 2015

@ventsislaf agreed, such a PR would be very welcome!

@ventsislaf ventsislaf mentioned this issue Feb 27, 2015
Closed
@tute tute closed this as completed in fce1401 Mar 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Configure ActiveJob's queue adapter to DelayedJob
3 participants
@tute @derekprior @ventsislaf