Set explain options to true

Author: dbernheisel

lib/ecto/adapters/sql.ex

@@ -432,6 +432,8 @@ defmodule Ecto.Adapters.SQL do
   end
 
   def explain(%{repo: repo} = adapter_meta, operation, queryable, opts) do
+    opts = conform_explain_opts(adapter_meta.adapter, opts)
+
     Ecto.Multi.new()
     |> Ecto.Multi.run(:explain, fn _, _ ->
       {prepared, prepared_params} = to_sql(operation, repo, queryable)
@@ -448,6 +450,17 @@ defmodule Ecto.Adapters.SQL do
     end
   end
 
+  @auto_true ~w[analyze verbose costs settings buffers timing summary plan]a
+  defp conform_explain_opts(Ecto.Adapters.Postgres, opts) do
+    Keyword.new(opts, fn
+      explain_opt when explain_opt in @auto_true ->
+        {explain_opt, true}
+      {opt, val} ->
+        {opt, val}
+    end)
+  end
+  defp conform_explain_opts(_, opts), do: opts
+
   @doc """
   Forces all connections in the repo pool to disconnect within the given interval.
 
@@ -952,8 +965,6 @@ defmodule Ecto.Adapters.SQL do
 
             * `:format`
 
-          All options except `format` are boolean valued and default to `false`.
-
           The allowed `format` values are `:map`, `:yaml`, and `:text`:
             * `:map` is the deserialized JSON encoding.
             * `:yaml` and `:text` return the result as a string.
@@ -970,9 +981,9 @@ defmodule Ecto.Adapters.SQL do
 
           """
 
-        _ -> :ok
+        _ ->
+          :ok
       end
-
       @spec explain(:all | :update_all | :delete_all, Ecto.Queryable.t(), opts :: Keyword.t()) ::
         String.t() | Exception.t() | list(map)
       def explain(operation, queryable, opts \\ []) do