Remove deprecated support to define explain in the connection adapter with 2 arguments

Author: rafaelfranca

activerecord/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Remove deprecated support to define `explain` in the connection adapter with 2 arguments.
+
+    *Rafael Mendonça França*
+
 *   Remove deprecated `ActiveRecord::ActiveJobRequiredError`.
 
     *Rafael Mendonça França*

activerecord/lib/active_record/explain.rb

@@ -24,7 +24,7 @@ def exec_explain(queries, options = []) # :nodoc:
           msg << binds.map { |attr| render_bind(attr) }.inspect
         end
         msg << "\n"
-        msg << connection_explain(sql, binds, options)
+        msg << connection.explain(sql, binds, options)
       end.join("\n")
 
       # Overriding inspect to be more human readable, especially in the console.
@@ -58,17 +58,5 @@ def build_explain_clause(options = [])
           "EXPLAIN for:"
         end
       end
-
-      def connection_explain(sql, binds, options)
-        if connection.method(:explain).parameters.size == 2
-          ActiveRecord.deprecator.warn(<<~MSG.squish)
-            The current database adapter, #{connection.adapter_name}, does not support explain options.
-            To remove this warning, the adapter must implement `build_explain_clause(options = [])`.
-          MSG
-          connection.explain(sql, binds)
-        else
-          connection.explain(sql, binds, options)
-        end
-      end
   end
 end

guides/source/7_2_release_notes.md

@@ -121,6 +121,8 @@ Please refer to the [Changelog][active-record] for detailed changes.
 
 *   Remove deprecated `ActiveRecord::ActiveJobRequiredError`.
 
+*   Remove deprecated support to define `explain` in the connection adapter with 2 arguments.
+
 
 ### Deprecations