diff options
| author | eileencodes <eileencodes@gmail.com> | 2015-04-04 09:17:06 -0400 |
|---|---|---|
| committer | eileencodes <eileencodes@gmail.com> | 2015-04-04 09:21:18 -0400 |
| commit | 191facc857bb4fb52078fb544c6bc1613a81cc80 (patch) | |
| tree | 0d5c7606114de1766a754e71d630becefc09552b /activerecord/lib/active_record/log_subscriber.rb | |
| parent | 5abadf1613710c6e6b54e8190b66543861711b8a (diff) | |
| download | rails-191facc857bb4fb52078fb544c6bc1613a81cc80.tar.gz rails-191facc857bb4fb52078fb544c6bc1613a81cc80.tar.bz2 rails-191facc857bb4fb52078fb544c6bc1613a81cc80.zip | |
Don't invoke sql_runtime if logger is not set to info
`sql_runtime` was getting invoked even when the logger was set to fatal.
This ensures that does not happen by checking that the logger is set to
info level before logging the view runtime.
This reduces the number of times `sql_runtime` is called for integration
tests with a fatal logger from 6 to 2.
Diffstat (limited to 'activerecord/lib/active_record/log_subscriber.rb')
| -rw-r--r-- | activerecord/lib/active_record/log_subscriber.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/log_subscriber.rb b/activerecord/lib/active_record/log_subscriber.rb index 6b26d7be78..af816a278e 100644 --- a/activerecord/lib/active_record/log_subscriber.rb +++ b/activerecord/lib/active_record/log_subscriber.rb @@ -31,9 +31,10 @@ module ActiveRecord end def sql(event) - self.class.runtime += event.duration return unless logger.debug? + self.class.runtime += event.duration + payload = event.payload return if IGNORE_PAYLOAD_NAMES.include?(payload[:name]) |