diff options
author | Xavier Noria <fxn@hashref.com> | 2011-12-02 09:19:13 -0800 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2011-12-02 09:19:13 -0800 |
commit | 36cc1f78eb17765f51e88085c30c39f40f3cb611 (patch) | |
tree | e15d7ddf662f1bb7612a9f952ffa8cde9e73c1aa /activerecord/lib/active_record | |
parent | 9acb340e88ac8a73ea410228e46457c3276fb1b4 (diff) | |
download | rails-36cc1f78eb17765f51e88085c30c39f40f3cb611.tar.gz rails-36cc1f78eb17765f51e88085c30c39f40f3cb611.tar.bz2 rails-36cc1f78eb17765f51e88085c30c39f40f3cb611.zip |
revises some details in the previous explain patch
Diffstat (limited to 'activerecord/lib/active_record')
-rw-r--r-- | activerecord/lib/active_record/explain.rb | 4 | ||||
-rw-r--r-- | activerecord/lib/active_record/relation.rb | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/explain.rb b/activerecord/lib/active_record/explain.rb index ab3a45ae53..4d013f0ef4 100644 --- a/activerecord/lib/active_record/explain.rb +++ b/activerecord/lib/active_record/explain.rb @@ -1,5 +1,5 @@ module ActiveRecord - module Explain # :nodoc + module Explain # :nodoc: # logging_query_plan calls could appear nested in the call stack. In # particular this happens when a relation fetches its records, since # that results in find_by_sql calls downwards. @@ -31,7 +31,7 @@ module ActiveRecord # SCHEMA queries cannot be EXPLAINed, also we do not want to run EXPLAIN on # our own EXPLAINs now matter how loopingly beautiful that would be. - SKIP_EXPLAIN_FOR = %(SCHEMA EXPLAIN) + SKIP_EXPLAIN_FOR = %w(SCHEMA EXPLAIN) def ignore_explain_notification?(payload) payload[:exception] || SKIP_EXPLAIN_FOR.include?(payload[:name]) end diff --git a/activerecord/lib/active_record/relation.rb b/activerecord/lib/active_record/relation.rb index e69e1aeaf8..1db26562b8 100644 --- a/activerecord/lib/active_record/relation.rb +++ b/activerecord/lib/active_record/relation.rb @@ -155,7 +155,7 @@ module ActiveRecord # Please see further details in the # {Active Record Query Interface guide}[http://edgeguides.rubyonrails.org/active_record_querying.html#running-explain]. def explain - results, sqls, binds = collecting_sqls_for_explain { exec_query } + _, sqls, binds = collecting_sqls_for_explain { exec_query } exec_explain(sqls, binds) end |