This project is archived and is in readonly mode.

Changeset [109d3ee38d1c39f0e27bc827065427635d6396b2] by Justin George

May 2nd, 2010 @ 09:45 PM

Make notifications go off even when an error is raised, so that we capture the underlying performance data [#4505 state:resolved]

This is important when trying to keep track of many layers of interrelated calls

i.e.:

ActiveRecord::Base.transaction do
MyModel.find(1) #ActiveRecord::NotFound end # should capture the full time until the error propagation

Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/109d3ee38d1c39f0e27bc827065427...

Committed by Justin George

  • M activesupport/lib/active_support/notifications/instrumenter.rb
  • M activesupport/test/notifications_test.rb

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>