clj-commons / clj-commons/metrics-clojure

3-arity version of `metrics.timers/timer` is not implemented

Open
#111 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Clojure
Stars
349
Forks
80
PR merge metrics
No merged PRs in 30d

Description

The arity-3 version of `metrics.timers/deftimer` tries to call `timer` with 3 arguments, which is not implemented. I guess it should either be dropped or call `timer-with-reservoir`. On a related note, is there a reason why `timer` passes the title through `metrics.core/metric-name` while `timer-with-reservoir` doesn't?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.