rails3 update_all 使用:order, :limit条件时，忽略条件问题

最新推荐文章于 2020-08-29 14:43:11 发布

原创最新推荐文章于 2020-08-29 14:43:11 发布 · 220 阅读

0 ·

CC 4.0 BY-SA版权

文章标签：

#ActiveRecord #Rails #Google

ruby&ror 专栏收录该内容

21 篇文章

订阅专栏

本文介绍了一个在delay_job中出现的Bug，该Bug导致失败任务被无限重试。通过对源代码进行修改，解决了update_all操作导致的任务状态判断错误问题。

这个bug是在使用delay_job（2.1.2）查出来的，后来google了一下，发现已经有人提交了这个问题

delay_job中如果有失败任务的话，后台会一直不停的运行失败任务

源代码修改，加注释的为原来的代码，注释后的为自己添加的修改代码

        # Reserve a single job in a single update query.  This causes workers to serialize on the
        # database and avoids contention.
        def self.reserve(worker, max_run_time = Worker.max_run_time)
#          affected_rows = 0
#          ::ActiveRecord::Base.silence do
#            affected_rows = jobs_available_to_worker(worker.name, max_run_time).limit(1).update_all(["locked_at = ?, locked_by = ?", db_time_now, worker.name])
#          end
#          if affected_rows == 1
#            locked_by_worker(worker.name, max_run_time).first
#          else
#            nil
#          end

＃修改代码start
          success = false
          ::ActiveRecord::Base.silence do
            available_job = jobs_available_to_worker(worker.name, max_run_time).first
            success = available_job.update_attributes(:locked_at => db_time_now, :locked_by => worker.name) if available_job
          end          
          if success
            locked_by_worker(worker.name, max_run_time).first
          else
            nil
          end
＃修改代码end
        end

引起的原因就是，那个update_all了。因为update_all前有limit处理，这样limit前的所有条件就会失效了，这样返回的结果就会始终有一个正在处理的任务。

［1］https://rails.lighthouseapp.com/projects/8994/tickets/6058-update_all-ignores-conditions-when-orders-and-limit-options-are-supplied