TL; DR：

Question

我有以下型号：

class User < ActiveRecord::Base
  def send_message(content)
    MessagePoro.new(content).deliver!
  end

  def self.send_to_all(content)
    threads = []
    all.each do |user|
      threads << Thread.new do
        user.send_message(content)
      end
    end
    threads.each(&:join)
  end
end

MessagePoro模型可以是简单的，例如应用程序/模型/ message_poro.rb：

class MessagePoro
  def initialize(content)
    # ...
  end

  def deliver!
    # ...
  end
end

现在，当我有例如100个用户，我正在运行User.send_to_all（“test”）我有时得到更严重的错误：

RuntimeError: Circular dependency detected while autoloading constant MessagePoro

或：

wrong number of arguments (1 for 0)

我认为一定是因为没有加载MessagePoro而且所有线程都试图同时加载它，或类似的东西。由于这些错误有时只会发生，我很确定只有当存在“竞争条件”或者有线程时才会有这些错误。我已经尝试在启动Threads之前初始化MessagePoro，并且我已经使用了eager_loading，但问题似乎仍然存在。还有什么可以尝试缓解这个问题？

Answer 1

我最近在尝试使用放置在[rails_root]/lib目录中的额外自定义库时遇到了一个非常类似的问题。

TL; DR：

您可以使用预先加载来解决问题，因为这可以确保在任何实际代码运行之前所有常量/模块/类都在内存中。但是为了这个工作：

您必须在Rails配置中设置config.eager_load = true（默认情况下，这在生产环境中已完成）
您要加载的课程所在的文件必须位于config.eager_load_paths，而不是config.autoload_paths。

OR

您可以使用require或require_dependency（另一个ActiveSupport功能）来确保在Rails自动加载之前显式加载您需要的代码。

额外说明：

Rails自动加载器完全独立于Ruby自动加载器 - 这似乎是因为Rails在尝试自动加载常量时对目录结构做了更多的推断。

不

Answer 2

这不是一个真正的答案，但我确实有更多的信息。抛出的错误来自ActiveSupport

 if file_path
    expanded = File.expand_path(file_path)
    expanded.sub!(/\.rb\z/, '')

    if loading.include?(expanded)
      raise "Circular dependency detected while autoloading constant #{qualified_name}"
    else
      require_or_load(expanded, qualified_name)
      raise LoadError, "Unable to autoload constant #{qualified_name}, expected #{file_path} to define it" unless from_mod.const_defined?(const_name, false)
      return from_mod.const_get(const_name)
    end
  elsif mod = autoload_module!(from_mod, const_name, qualified_name, path_suffix)
    return mod
  elsif

经过进一步研究，我们可以看到loading是一个类变量。

# Stack of files being loaded.
mattr_accessor :loading
self.loading = []

检查同一文件的两个线程：

第一个线程点击此代码并将路径放入加载

      loading << expanded

然后第二个线程检查扩展和命中

表示的路径

 if loading.include?(expanded)
      raise "Circular dependency detected while autoloading constant #{qualified_name}"

我错过了什么？ ActiveSupport :: Dependencies不是线程安全的吗？

Answer 3

经过一番研究后发现，自动加载现在是线程安全的。所以它可能是回归。结帐Threading with the AWS SDK for Ruby。该补丁由Charles Nutter在ruby 2.0.0 autoload is not thread-safe

中介绍

无论如何只有这个类，你可以通过手动要求来避免自动加载它。只需手动使用它。

require 'message_poro'
class User
def self.send_to_all(content) 
  ... 
end

Rails 4.2自动加载不是线程安全的

3 个答案:

TL; DR：

更多信息

额外说明：