Different identifier for increment and timing
We have some problems to visualize the metrics as increment and timing functions are called with the same identifier. Adding '.counter' and '.timer' suffixes will differentiate them
Coverage increased (+0.3%) to 96.429% when pulling cadf647bf78fa586854e2ed5d8169885cbb9c921 on xarlio:patch-1 into 031723a2c0925fcde69ebf936a1752f5d1ec3461 on mac-:master.
Thanks for the contribution! A couple of questions:
- What tool are you using to visualize the metrics? From my experience, Graphite and Grafana are able to differentiate these metrics without changing the metric name.
- I don't have a problem with allowing a way to differentiate these by modifying the metric names, however, instead of hard-coding the
.timerand.countersuffixes, can you make it configurable via options so that we don't force everyone into this convention?
We are using Datadog, and AFAIK there is no way to differentiate the two metrics. I will make it optional. Any suggestion for the options names and its defaults?
you could do something as simple as a boolean like includeMetricTypeInName as an option, and if it's true, add it like you are doing now.