{"action":"created","issue":{"url":"https://api.github.com/repos/ratpack/ratpack/issues/756","labels_url":"https://api.github.com/repos/ratpack/ratpack/issues/756/labels{/name}","comments_url":"https://api.github.com/repos/ratpack/ratpack/issues/756/comments","events_url":"https://api.github.com/repos/ratpack/ratpack/issues/756/events","html_url":"https://github.com/ratpack/ratpack/issues/756","id":101286606,"number":756,"title":"Add option to disable default metric paths in ratpack-codahale-metrics","user":{"login":"drmaas","id":3291796,"avatar_url":"https://avatars.githubusercontent.com/u/3291796?v=3","gravatar_id":"","url":"https://api.github.com/users/drmaas","html_url":"https://github.com/drmaas","followers_url":"https://api.github.com/users/drmaas/followers","following_url":"https://api.github.com/users/drmaas/following{/other_user}","gists_url":"https://api.github.com/users/drmaas/gists{/gist_id}","starred_url":"https://api.github.com/users/drmaas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drmaas/subscriptions","organizations_url":"https://api.github.com/users/drmaas/orgs","repos_url":"https://api.github.com/users/drmaas/repos","events_url":"https://api.github.com/users/drmaas/events{/privacy}","received_events_url":"https://api.github.com/users/drmaas/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"rhart","id":226023,"avatar_url":"https://avatars.githubusercontent.com/u/226023?v=3","gravatar_id":"","url":"https://api.github.com/users/rhart","html_url":"https://github.com/rhart","followers_url":"https://api.github.com/users/rhart/followers","following_url":"https://api.github.com/users/rhart/following{/other_user}","gists_url":"https://api.github.com/users/rhart/gists{/gist_id}","starred_url":"https://api.github.com/users/rhart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhart/subscriptions","organizations_url":"https://api.github.com/users/rhart/orgs","repos_url":"https://api.github.com/users/rhart/repos","events_url":"https://api.github.com/users/rhart/events{/privacy}","received_events_url":"https://api.github.com/users/rhart/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2015-08-16T18:16:01Z","updated_at":"2015-08-17T18:54:48Z","closed_at":null,"body":"When this module is installed, by default all unique paths result in a new metric. For large data sets the number of metrics becomes unbounded. If we're sending metrics to external systems like statsd, the network traffic becomes unmanageable. \r\n\r\nI know there is a way of collapsing metrics by group name, but the default metric handler/interceptor still execute. It would be useful to provide an option to disable the default metric handler/interceptor so that we could drop in our own handler/interceptor, while taking advantage of the other features provided by the module."},"comment":{"url":"https://api.github.com/repos/ratpack/ratpack/issues/comments/131927604","html_url":"https://github.com/ratpack/ratpack/issues/756#issuecomment-131927604","issue_url":"https://api.github.com/repos/ratpack/ratpack/issues/756","id":131927604,"user":{"login":"rhart","id":226023,"avatar_url":"https://avatars.githubusercontent.com/u/226023?v=3","gravatar_id":"","url":"https://api.github.com/users/rhart","html_url":"https://github.com/rhart","followers_url":"https://api.github.com/users/rhart/followers","following_url":"https://api.github.com/users/rhart/following{/other_user}","gists_url":"https://api.github.com/users/rhart/gists{/gist_id}","starred_url":"https://api.github.com/users/rhart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhart/subscriptions","organizations_url":"https://api.github.com/users/rhart/orgs","repos_url":"https://api.github.com/users/rhart/repos","events_url":"https://api.github.com/users/rhart/events{/privacy}","received_events_url":"https://api.github.com/users/rhart/received_events","type":"User","site_admin":false},"created_at":"2015-08-17T18:54:48Z","updated_at":"2015-08-17T18:54:48Z","body":"Sure, makes sense.  I'd still like to understand this a little deeper though.  To see whether there is anything else we can be doing.\r\n\r\n@drmaas are you saying that the metric collapsing solves your network traffic issues but you don't want every request to go through the handler?  Even if we allowed requests to be excluded at the handler/interceptor level all requests would still go through the handler, just not all recorded.  But would adding this functionality help?  What would your handler/interceptor do differnet?"}}