benchmark DB time should calculate all DB connections instead of only the default one.

description

benchmark DB time, rails only considered the default DB connection(in multiple DB situation).

details

check http://github.com/rails/rails/tr... line 55, 101

it only picked the run timing of the default DB connection:

ActionRecord::Base.connection.reset_runtime

thus, any other connections(i.e. connection to my reporting DB) is wrongly ignored. the result is: in production.log, you will see Rendering time nearly 100%, while DB time 0%.

solution

define ActiveRecord::Base::reset_db_runtime

def self.reset_db_runtime

  timing = 0
  self.active_connections.each_value do |conn|
    timing += conn.reset_runtime
  end
  timing

end

replace every call in benchmarking.rb ActiveRecord::Base.connection.reset_runtime to ActiveRecord::Base.reset_db_runtime

others

I am using rails-1.2.3, but it seems the latest code still have the issue.