Skip to content
This repository has been archived by the owner on Oct 5, 2018. It is now read-only.

Version 3.0 #179

Closed
6 tasks done
ScotterC opened this issue Apr 29, 2016 · 6 comments
Closed
6 tasks done

Version 3.0 #179

ScotterC opened this issue Apr 29, 2016 · 6 comments

Comments

@ScotterC
Copy link
Collaborator

ScotterC commented Apr 29, 2016

Path

For Version 3

  • Remove all adapters except AJ
  • Bump required Ruby version to 2.0
  • Bump required Rails version to 4.2
  • Refactoring
  • Cleanup Readme

For Version 2.10

Anything else?

@morgoth
Copy link
Collaborator

morgoth commented May 2, 2016

I think nothing more for 2.10

@ScotterC
Copy link
Collaborator Author

ScotterC commented May 2, 2016

@morgoth cool. We'll leave the issue as is then 👍

morgoth added a commit that referenced this issue May 3, 2016
morgoth added a commit that referenced this issue May 3, 2016
@dwbutler
Copy link

I'm using the Delayed Job adapter because ActiveJob does not (yet) support setting job priority. We need image processing jobs to be enqueued with highest priority.

It would be great if the removed adapters could be extracted to separate gems instead of being removed entirely.

@morgoth
Copy link
Collaborator

morgoth commented Jul 18, 2016

@dwbutler I think it would require even more effort to maintain separate gems.

Do you think you could solve your issue in a little different way, like having 2 background processing serwers/pods/instances that process different queues, so one of them would process only important ones?

script/delayed_job --queue=critical start
script/delayed_job --queues=default,mailers,all-others start

morgoth added a commit that referenced this issue Jul 18, 2016
morgoth added a commit that referenced this issue Jul 18, 2016
@dwbutler
Copy link

Yes, the issue can be worked around in various ways.

The main problem, in my mind, is that ActiveJob will probably never support the full range of features in the different queueing systems. So only supporting ActiveJob is a step backwards.

I can only hope that in the future, ActiveJob will give better configuration control into the underlying implementation instead of forcing everyone to use a one-size-fits-all interface.

@jrgifford
Copy link
Owner

jrgifford commented Aug 1, 2016

👍

morygonzalez pushed a commit to morygonzalez/delayed_paperclip that referenced this issue Sep 28, 2016
morygonzalez pushed a commit to morygonzalez/delayed_paperclip that referenced this issue Sep 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants