umgeher/leveled
0
Fork 0
Code Issues Pull requests Releases Activity Actions

Allow db_id prefix to be empty when undefined (#464)

Browse source 
There is a db_id of 0 (for example for vnode 0 in a riak cluster) - so naming databases without an id 0 creates confusion
This commit is contained in:
Martin Sumner 2024-12-04 13:22:39 +00:00 • committed by GitHub
parent 69e8b29d1f
commit fe9710b47a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/leveled_log.erl
View file

@ -41,7 +41,7 @@
-record(log_options,
{log_level = info :: log_level(),
forced_logs = [] :: [atom()],
database_id = 0 :: non_neg_integer()}).
database_id :: non_neg_integer()|undefined}).
-type log_level() :: debug | info | warning | error | critical.
-type log_options() :: #log_options{}.
@ -490,11 +490,15 @@ log_randomtimer(LogReference, Subs, StartTime, RandomProb) ->
ok
end.
-spec log_prefix(atom(), non_neg_integer(), pid()) -> io_lib:chars().
-spec log_prefix(atom(), non_neg_integer()|undefined, pid()) -> io_lib:chars().
log_prefix(LogRef, undefined, Pid) ->
["log_ref=", atom_to_list(LogRef), " pid=", pid_to_list(Pid), " "];
log_prefix(LogRef, DBid, Pid) ->
["log_ref=", atom_to_list(LogRef),
[
"log_ref=", atom_to_list(LogRef),
" db_id=", integer_to_list(DBid),
" pid=", pid_to_list(Pid), " "].
" pid=", pid_to_list(Pid), " "
].
-spec duration_text(erlang:timestamp()) -> io_lib:chars().
duration_text(StartTime) ->