Dempsy / dempsy

Distributed Elastic Message Processing System
Apache License 2.0
195 stars 21 forks source link

adaptor showing up under graphite - under local folder #66

Closed mmccuiston closed 12 years ago

mmccuiston commented 12 years ago

When we view the graphite dashboard we find that the ProbeAdaptor shows up in a location marked "local" instead of a location using the client machines' IP address.

However being that the Probe Adaptor is not an MP - the code in

StatsCollectorFactoryCoda::buildPrefix

line 146: if (destination != null && destination instanceof TcpDestination)

Causes the prefix to default to "local" and the ProbeAdaptor stats wind up in the default "local" bucket which is confusing to the QA folks.

jimfcarroll commented 12 years ago

Thanks for digging into this and finding the exact location. It should be an easy fix.

jimfcarroll commented 12 years ago

Closed by PR64