将块传递给delayed_job

时间:2012-08-13 12:32:24

标签: ruby-on-rails ruby block delayed-job

我有一个标记为由delayed_job异步处理的函数:

class CapJobs
  def execute(params, id)
    begin
      unless Rails.env == "test"
        Capistrano::CLI.parse(params).execute!
      end
    rescue
      site = Site.find(id)
      site.records.create!(:date => DateTime.now, :action => "Task Failure: #{params[0]}", :type => :failure)    
      site.save
    ensure
      yield id
    end 
  end   
  handle_asynchronously :execute
end 

当我运行这个函数时,我传入一个块:

 capjobs = CapJobs.new
 capjobs.execute(parameters, @site.id) do |id|
   asite = Site.find(id)
   asite.records.create!(:date => DateTime.now, :action => "Created", :type => :init)
   asite.status = "On Demo"
   asite.dev = true
   asite.save
 end

这在没有delayed_job的情况下运行时效果很好但是当运行时我得到以下错误

2012-08-13T09:24:36-0300: [Worker(delayed_job host:eagle pid:12089)] SitesHelper::CapJobs#execute_without_delay failed with LocalJumpError: no block given (yield) - 0 failed attempts
2012-08-13T09:24:36-0300: [Worker(delayed_job host:eagle pid:12089)] PERMANENTLY removing SitesHelper::CapJobs#execute_without_delay because of 1 consecutive failures.
2012-08-13T09:24:36-0300: [Worker(delayed_job host:eagle pid:12089)] 1 jobs processed at 0.0572 j/s, 1 failed ...

似乎没有拿起传入的块。这不是正确的方法吗?或者我应该找到不同的方法?

3 个答案:

答案 0 :(得分:9)

delayed_job的工作原理是将作业保存到数据存储(通常是主数据库)中,然后在后台进程中将作业加载到此数据存储中,然后处理/执行。

要将作业保存到数据库中,delayed_job需要以某种方式保存使用什么参数调用哪个对象的方法。这是通过将所有内容序列化为字符串来完成的(delayed_job使用 yaml )。不幸的是,块无法序列化。所以后台工作者不知道块参数,并在没有它的情况下调用该方法。当方法试图屈服于块时,这会导致LocalJumpError

答案 1 :(得分:1)

我找到了一种方法。它有点hacky但效果很好。我发现this article讨论了创建SerializableProc类的问题。如果我把它传递给函数,那么一切都很好。

答案 2 :(得分:0)

大多数人会将此视为抽象问题。

proc代码可能没有从run-to-run(vars除外)改变,因此你应该将块代码变成类或实例方法。传递该方法的名称,然后在execute方法中调用它,例如

@some_data = CapJobs.send( target_method )

或者甚至更好 - 甚至

@some_data = DomainSpecificModel.send( target_method )