Need to include a meter registry short name in our config property names
donbourne opened this issue · 2 comments
donbourne commented
I think our config property names need to be more specific. right now we prepend "mp.metrics." to whatever the meter registry property name is. Graphite, as an example, asks for "host" and "port". So we end up with...
mp.metrics.host = ...
mp.metrics.port= ...
... that would be fine if you only support having one meter registry installed at a time, but if you have more than one you need the meter registry name in the prefix. For example, we could have...
mp.metrics.graphite.host=...
mp.metrics.graphite.port=...
donbourne commented
@jgallimore , WDYT on this one? is the extra short name needed?
donbourne commented
looks like this already exists in the spec. not sure how I missed that before!