summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorIvan Tashkinov <ivantashkinov@gmail.com>2021-12-19 20:35:00 +0300
committerIvan Tashkinov <ivantashkinov@gmail.com>2021-12-19 20:45:28 +0300
commite009950845c6d1e7864bb68ea1258c58438ee3aa (patch)
treea93ca31b8fddfa294e4e7d1c9aee51b79818a1d6 /lib
parent949a53e327fa2d4ca2099cd4ca6fa2e3fd9e789a (diff)
Slow queries logging improvements: added EXPLAIN results, listed params, improved stacktrace.log-slow-queries
Diffstat (limited to 'lib')
-rw-r--r--lib/pleroma/telemetry/logger.ex20
1 files changed, 15 insertions, 5 deletions
diff --git a/lib/pleroma/telemetry/logger.ex b/lib/pleroma/telemetry/logger.ex
index 1dea13acd..c079f34f2 100644
--- a/lib/pleroma/telemetry/logger.ex
+++ b/lib/pleroma/telemetry/logger.ex
@@ -101,13 +101,19 @@ defmodule Pleroma.Telemetry.Logger do
def handle_event(
[:pleroma, :repo, :query] = _name,
%{query_time: query_time} = _measurements,
- %{source: source, query: query} = _metadata,
+ %{source: source, query: query, params: query_params, repo: repo} = _metadata,
_config
)
when query_time > 500_000 and source not in [nil, "oban_jobs"] do
{:current_stacktrace, stacktrace} = Process.info(self(), :current_stacktrace)
- stacktrace =
+ sql_explain =
+ with {:ok, %{rows: explain_result_rows}} <-
+ repo.query("EXPLAIN " <> query, query_params, log: false) do
+ Enum.map_join(explain_result_rows, "\n", & &1)
+ end
+
+ pleroma_stacktrace =
Enum.filter(stacktrace, fn
{__MODULE__, _, _, _} ->
false
@@ -120,13 +126,17 @@ defmodule Pleroma.Telemetry.Logger do
Logger.warn(fn ->
"""
- Query took longer than 500ms!
+ Slow query!
Total time: #{query_time / 1_000}ms
- #{inspect(query)}
+ #{query}
+
+ #{inspect(query_params)}
+
+ #{sql_explain}
- #{inspect(stacktrace, pretty: true)}
+ #{Exception.format_stacktrace(pleroma_stacktrace)}
"""
end)
end