{"action":"created","issue":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/2342","labels_url":"https://api.github.com/repos/influxdb/influxdb/issues/2342/labels{/name}","comments_url":"https://api.github.com/repos/influxdb/influxdb/issues/2342/comments","events_url":"https://api.github.com/repos/influxdb/influxdb/issues/2342/events","html_url":"https://github.com/influxdb/influxdb/pull/2342","id":69433485,"number":2342,"title":"graphite: log the field name when we encounter a value parse failure","user":{"login":"segfault","id":3042,"avatar_url":"https://avatars.githubusercontent.com/u/3042?v=3","gravatar_id":"","url":"https://api.github.com/users/segfault","html_url":"https://github.com/segfault","followers_url":"https://api.github.com/users/segfault/followers","following_url":"https://api.github.com/users/segfault/following{/other_user}","gists_url":"https://api.github.com/users/segfault/gists{/gist_id}","starred_url":"https://api.github.com/users/segfault/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/segfault/subscriptions","organizations_url":"https://api.github.com/users/segfault/orgs","repos_url":"https://api.github.com/users/segfault/repos","events_url":"https://api.github.com/users/segfault/events{/privacy}","received_events_url":"https://api.github.com/users/segfault/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-04-19T15:06:55Z","updated_at":"2015-04-21T04:13:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdb/influxdb/pulls/2342","html_url":"https://github.com/influxdb/influxdb/pull/2342","diff_url":"https://github.com/influxdb/influxdb/pull/2342.diff","patch_url":"https://github.com/influxdb/influxdb/pull/2342.patch"},"body":"The java metrics library will dump anything and everything to graphite leading to many error messages like:\r\n\r\n```\r\n[graphite] 2015/04/19 10:11:46 unable to parse data: strconv.ParseFloat: parsing \"[J@4deff1ac\": invalid syntax\r\n[graphite] 2015/04/19 10:11:46 unable to parse data: strconv.ParseFloat: parsing \"[J@5b8c4bb5\": invalid syntax\r\n```\r\n\r\nIt 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.\r\nThe field name can then be used to filter the offending field from the stats publish.\r\n\r\nThe ParseFloat related error messages have been modified such they include the field name and the value that was being parsed (value|timestamp).\r\n\r\nThe result should look something like:\r\n```\r\n[graphite] 2015/04/19 11:03:16 unable to parse data: field \"host/cas-nyc1/.org.apache.cassandra.metrics.ColumnFamily.bitcrest.depth.EstimatedColumnCountHistogram.value\" value: strconv.ParseFloat: parsing \"[J@6d603059\": invalid syntax\r\n[graphite] 2015/04/19 11:03:16 unable to parse data: field \"host/cas-nyc2/.org.apache.cassandra.metrics.ColumnFamily.bitcrest.depth.EstimatedRowSizeHistogram.value\" value: strconv.ParseFloat: parsing \"[J@593b8406\": invalid syntax\r\n```\r\nWhich provides enough information to fix the problem at it's source."},"comment":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/comments/94633160","html_url":"https://github.com/influxdb/influxdb/pull/2342#issuecomment-94633160","issue_url":"https://api.github.com/repos/influxdb/influxdb/issues/2342","id":94633160,"user":{"login":"otoolep","id":536312,"avatar_url":"https://avatars.githubusercontent.com/u/536312?v=3","gravatar_id":"","url":"https://api.github.com/users/otoolep","html_url":"https://github.com/otoolep","followers_url":"https://api.github.com/users/otoolep/followers","following_url":"https://api.github.com/users/otoolep/following{/other_user}","gists_url":"https://api.github.com/users/otoolep/gists{/gist_id}","starred_url":"https://api.github.com/users/otoolep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/otoolep/subscriptions","organizations_url":"https://api.github.com/users/otoolep/orgs","repos_url":"https://api.github.com/users/otoolep/repos","events_url":"https://api.github.com/users/otoolep/events{/privacy}","received_events_url":"https://api.github.com/users/otoolep/received_events","type":"User","site_admin":false},"created_at":"2015-04-21T04:13:33Z","updated_at":"2015-04-21T04:13:33Z","body":"Can you please rebase with our master? There were some issues with our CI build script."}}