jrgifford/delayed_paperclip

Add simple queue support for delayed_job

Closed this issue · 5 comments

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.

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
scott@artsicle.com
C. 914 400 5085

On Tue, Jul 9, 2013 at 10:37 AM, KieranBroadfoot
notifications@github.comwrote:

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
.

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.

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. 😦

Looks like it's fixed in #169