emqx / esockd / 592

67%

master: 67%

Pull #227

github

refactor: merge sock-error counters into get_stats/1

esockd_server:get_stats/1 has no production callers in this tree (only
tests), so the per-reason socket-error counters can safely live in its
result instead of a separate get_sock_errors/1 query.

Store them under the same {{Proto, ListenOn}, Metric} key shape with the
metric {sock_error, Reason}; get_stats/1 now returns everything for a
listener, e.g. [{accepted, 2}, {discarded, 1},
{{sock_error, enotconn}, 5}].  del_stats needs one match_delete less.
get_sock_errors/1 is removed; inc_sock_error/2 stays as the write API.

31 of 57 new or added lines in 4 files covered. (54.39%)

755 of 1100 relevant lines covered (68.64%)

202.0 hits per line