Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fixed that benchmarking times for rendering included db runtimes #987
…[skaes@web.de]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1070 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information
dhh committed Apr 2, 2005
1 parent 7c8d2f2 commit 91834e0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
14 changes: 10 additions & 4 deletions actionpack/lib/action_controller/benchmarking.rb
Expand Up @@ -19,7 +19,11 @@ def render_with_benchmark(template_name = default_template_name, status = "200 O
if logger.nil?
render_without_benchmark(template_name, status)
else
db_runtime = ActiveRecord::Base.connection.reset_runtime
@rendering_runtime = Benchmark::measure{ render_without_benchmark(template_name, status) }.real
@db_rt_before_render = db_runtime
@db_rt_after_render = ActiveRecord::Base.connection.reset_runtime
@rendering_runtime -= @db_rt_after_render
end
end

Expand All @@ -28,7 +32,7 @@ def perform_action_with_benchmark
perform_action_without_benchmark
else
runtime = [Benchmark::measure{ perform_action_without_benchmark }.real, 0.0001].max
log_message = "Completed in #{sprintf("%4f", runtime)} (#{(1 / runtime).floor} reqs/sec)"
log_message = "Completed in #{sprintf("%.5f", runtime)} (#{(1 / runtime).floor} reqs/sec)"
log_message << rendering_runtime(runtime) if @rendering_runtime
log_message << active_record_runtime(runtime) if Object.const_defined?("ActiveRecord") && ActiveRecord::Base.connected?
logger.info(log_message)
Expand All @@ -37,13 +41,15 @@ def perform_action_with_benchmark

private
def rendering_runtime(runtime)
" | Rendering: #{sprintf("%f", @rendering_runtime)} (#{sprintf("%d", (@rendering_runtime / runtime) * 100)}%)"
" | Rendering: #{sprintf("%.5f", @rendering_runtime)} (#{sprintf("%d", (@rendering_runtime * 100) / runtime)}%)"
end

def active_record_runtime(runtime)
db_runtime = ActiveRecord::Base.connection.reset_runtime
db_percentage = (db_runtime / runtime) * 100
" | DB: #{sprintf("%f", db_runtime)} (#{sprintf("%d", db_percentage)}%)"
db_runtime += @db_rt_before_render if @db_rt_before_render
db_runtime += @db_rt_after_render if @db_rt_after_render
db_percentage = (db_runtime * 100) / runtime
" | DB: #{sprintf("%.5f", db_runtime)} (#{sprintf("%d", db_percentage)}%)"
end
end
end
2 changes: 2 additions & 0 deletions activerecord/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*

* Fixed that benchmarking times for rendering included db runtimes #987 [skaes@web.de]

* Fixed boolean queries for t/f fields in PostgreSQL #995 [dave@cherryville.org]

* Added that model.items.delete(child) will delete the child, not just set the foreign key to nil, if the child is dependent on the model #978 [bitsweat]
Expand Down

0 comments on commit 91834e0

Please sign in to comment.