Allow to set queue name for background jobs

jrgifford / delayed_paperclip

Process your Paperclip attachments in the background with delayed_job or Resque.

MIT License

402 stars 155 forks source link

Closed morgoth closed 8 years ago

morgoth commented 8 years ago

@ScotterC This might be an incompatible change for some users.

Previously delayed_job adapter queue name was nil - now defaults to "paperclip"
Setting queue name changed a little in each adapter, so it may affect users that already changed it by hand, in example in AJ adapter it was possible before by:

# Overwrite queue name which was "paperclip"
class DelayedPaperclip::Jobs::ActiveJob
  queue_as :default
end

which will no longer work now.

morgoth commented 8 years ago

morgoth commented 8 years ago

@ScotterC What do you think about this? As there are lot of changes on master, maybe it could be a part of 3.0 release?

ScotterC commented 8 years ago

@morgoth I haven't had a chance to give this a thorough review but a 3.0 release sounds like a good idea. Will give it some time later this week.

morgoth commented 8 years ago

Any chance to move it forward?

ScotterC commented 8 years ago

👍