Rails迁移批量回填用户字段报NoMethodError如何解决

报错来自对in_batches返回值的误解：迭代批次时传入块的参数是ActiveRecord::Relation类型的查询范围对象，代表当前批次的查询条件，不是单个用户模型，也不是用户实例数组。这个对象本身没有country实例方法，调用users.country自然会抛出NoMethodError。
原代码还有逻辑层面的问题：就算不触发报错，update_all传键值对的写法是把所有匹配记录的字段更新为同一个固定值，根本实现不了「把每条记录自身的country字段值复制到同记录passport_country字段」的回填需求。

生产环境海量数据的字段回填，不要在Ruby层加载模型实例做循环更新，直接在SQL层面完成字段值拷贝，性能最高，对数据库压力最小：

• 把update_all的入参改为SQL片段，直接让数据库执行同记录字段赋值，不需要实例化任何用户模型
• 建议修正原迁移类名的拼写错误（原类名里把Country拼成了County），避免后续维护产生歧义
• 可以补充down方法，支持迁移回滚

修正后的完整迁移代码如下：

class CopyUsersCountryToPassportCountry < ActiveRecord::Migration[6.0]
  disable_ddl_transaction!

  def up
    User.unscoped.where.not(country: nil).in_batches do |batch|
      # 生成的SQL逻辑：更新当前批次所有记录，把passport_country设为同条记录的country值
      batch.update_all("passport_country = country")
      sleep(0.01) # 批次间留短暂间隔，避免持续占满数据库连接和IO资源
    end
  end

  def down
    User.unscoped.where.not(passport_country: nil).in_batches do |batch|
      batch.update_all(passport_country: nil)
      sleep(0.01)
    end
  end
end

注意：绝对不要在批次块里写batch.each { |u| u.update(passport_country: u.country) }这类遍历单条记录更新的逻辑，这种写法会把整批用户记录加载到应用内存，产生大量单行更新SQL，数据量到百万级以上就很容易打满数据库CPU、触发应用OOM，完全不适合生产环境使用。

内容的提问来源于stack exchange，提问作者Clayton Siby