emqx / esockd / 586

67%

master: 67%

Build Type

push

github

Committed by terry-xiaoyu

Commit Message

refactor: keep all listener counters in one stats table, drop init_stats

Review feedback: a second ETS table for socket-error counters is
unnecessary, and init_stats is redundant because ets:update_counter/4
already creates missing counters with a default of 0.

- sock-error reasons now live in esockd_stats under the namespaced key
  {sock_error, Proto, ListenOn, Reason}, which get_stats/1 naturally
  excludes (its match only picks up {{Proto, ListenOn}, Metric} keys);
  get_sock_errors/1 and del_stats match the sock_error namespace
- remove init_stats/2 (export, handle_call and call sites in
  esockd_listener_sup and stats_fun); counters appear lazily on first
  update
- t_get_stats no longer expects {discarded, 0} since metrics are only
  present once incremented

Coverage Stats

5 of 5 new or added lines in 1 file covered. (100.0%)

52 existing lines in 4 files now uncovered.

748 of 1087 relevant lines covered (68.81%)

203.77 hits per line