The java metrics library will dump anything and everything to graphite leading to many error messages like:
[graphite] 2015/04/19 10:11:46 unable to parse data: strconv.ParseFloat: parsing "[J@4deff1ac": invalid syntax
[graphite] 2015/04/19 10:11:46 unable to parse data: strconv.ParseFloat: parsing "[J@5b8c4bb5": invalid syntax
It makes sense to include the name field given that the graphite parsing code has already validated the name and tags by the time the value is parsed.
The field name can then be used to filter the offending field from the stats publish.
The ParseFloat related error messages have been modified such they include the field name and the value that was being parsed (value|timestamp).
This encapsulates a few previously free-floating methods. This means
that params such as "separator" and "last-enabled" are not passed from
function to function, and also makes initialization of TCP and UDP
Servers somewhat clearer.
It also removes early use of "GraphiteProcessor".
Added ability to specifiy a seperator for a metric_path.
Default is "."
Added ability to specify position of name from metric_path
Options are "first" or "last".
Defaults to the first position.
Normalized the graphite.Metric struct to only have a Value.
Changes to be committed:
modified: cmd/influxd/config.go
modified: cmd/influxd/config_test.go
modified: cmd/influxd/run.go
modified: graphite/graphite.go
modified: graphite/graphite_test.go
Mark Guzman
Cory LaNou
Philip O'Toole
Philip O'Toole
David Norton
Ben Johnson