search

reinh / statsd

A Ruby Statsd client that isn't a direct port of the Python example code. Because Ruby isn't Python.
MIT License
411 stars 154 forks source link

Arbitrary metric name parsing #58

Closed erez-rabih closed 9 years ago

erez-rabih commented 9 years ago

Hi,

In line 387 of file lib/statsd.rb we replace "::" with "." to have directory structure which resembles the ruby module scoping.

While I understand the logic behind this decision, I'd like to propose to make it configurable and not mandatory. I for example, would like to replace "::" with "-" and not "." to have flat metric hierarchy.

If this idea is accepted I'm also willing to make a PR.

I'd be happy to hear your thoughts.

raggi commented 9 years ago

The most common graphite setups use a period for directory separation, but making this configurable seems reasonable.