Merge pull request #504 from rabbitmq/reduce-noise

Only log ETS delete time if it's more than 25ms
rabbitmq · Jan 31, 2025 · 77230dd · 77230dd
2 parents 130c01a + bf362e3
commit 77230dd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/ra_log_ets.erl b/src/ra_log_ets.erl
@@ -147,7 +147,7 @@ handle_cast({exec_delete, UId, Spec},
     %% TODO: delete from ra_log_open_mem_tables if {delete, tid()}
     try timer:tc(fun () -> ra_mt:delete(Spec) end) of
         {Time, Num} ->
-            ?DEBUG("ra_log_ets: ra_mt:delete/1 took ~bms to delete ~w ~b entries",
+            ?DEBUG_IF(Time > 25_000, "ra_log_ets: ra_mt:delete/1 took ~bms to delete ~w ~b entries",
                    [Time div 1000, Spec, Num]),
             ok
     catch
@@ -166,7 +166,7 @@ handle_cast({delete_mem_tables, UId},
     [begin
          try timer:tc(fun () -> ets_delete(Tid) end) of
              {Time, true} ->
-                 ?DEBUG("ra_log_ets: ets:delete/1 took ~bms to delete ~w",
+                 ?DEBUG_IF(Time > 25_000, "ra_log_ets: ets:delete/1 took ~bms to delete ~w",
                         [Time div 1000, Tid]),
                  ok
          catch