Add simple queue support for delayed_job about delayed_paperclip HOT 5 CLOSED

I'm not super familiar with delayed job but if the feature can be added
unobtrusively then I'd love to have the pull request.

Scott Carleton
Artsicle, Co-Founder / CTO
[email protected]
C. 914 400 5085

On Tue, Jul 9, 2013 at 10:37 AM, KieranBroadfoot
[email protected]:

I would like to suggest the inclusion of queuing semantics into
jobs/delayed_job.rb for those who use the latest versions of DJ. Happy to
send a very simple pull request if you would be interested.

This enables me (and anyone else who would benefit) to distinctly manage
delayed paperclip tasks without needing to ensure all other tasks are kept
off the main queue.

Thanks
k.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/41
.

from delayed_paperclip.

The change is very straightforward (:queue => :paperclip) which I would be happy to provide. However I don't have a 2.X DJ install to hand to ensure it will not break anything. After a little investigation I suspect AR will throw a fit and hence this change does not make sense if you wish to continue supporting that older version. If anyone listening has an install of 2.X to help me test it would be great to confirm.

from delayed_paperclip.

I unfortunately do not have a DJ 2.x install, and I unfortunately have to agree that it's going to give AR a fit. 😦

from delayed_paperclip.

+1

from delayed_paperclip.

Looks like it's fixed in #169

from delayed_paperclip.