{"action":"created","issue":{"url":"https://api.github.com/repos/spotify/semantic-metrics/issues/4","repository_url":"https://api.github.com/repos/spotify/semantic-metrics","labels_url":"https://api.github.com/repos/spotify/semantic-metrics/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/spotify/semantic-metrics/issues/4/comments","events_url":"https://api.github.com/repos/spotify/semantic-metrics/issues/4/events","html_url":"https://github.com/spotify/semantic-metrics/pull/4","id":159434471,"number":4,"title":"Initial support for remote metrics","user":{"login":"liljencrantz","id":212675,"avatar_url":"https://avatars.githubusercontent.com/u/212675?v=3","gravatar_id":"","url":"https://api.github.com/users/liljencrantz","html_url":"https://github.com/liljencrantz","followers_url":"https://api.github.com/users/liljencrantz/followers","following_url":"https://api.github.com/users/liljencrantz/following{/other_user}","gists_url":"https://api.github.com/users/liljencrantz/gists{/gist_id}","starred_url":"https://api.github.com/users/liljencrantz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liljencrantz/subscriptions","organizations_url":"https://api.github.com/users/liljencrantz/orgs","repos_url":"https://api.github.com/users/liljencrantz/repos","events_url":"https://api.github.com/users/liljencrantz/events{/privacy}","received_events_url":"https://api.github.com/users/liljencrantz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-09T15:17:28Z","updated_at":"2016-06-09T16:34:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spotify/semantic-metrics/pulls/4","html_url":"https://github.com/spotify/semantic-metrics/pull/4","diff_url":"https://github.com/spotify/semantic-metrics/pull/4.diff","patch_url":"https://github.com/spotify/semantic-metrics/pull/4.patch"},"body":"This patch makes it possible to transmit\r\nmetrics to a remote SemanticAggregator\r\ninstance via http. It emulates the plain\r\nSemanticMetrics API as much as possible.\r\nCurrently only supports meters, and no\r\nsupport for HA via sharding."},"comment":{"url":"https://api.github.com/repos/spotify/semantic-metrics/issues/comments/224951827","html_url":"https://github.com/spotify/semantic-metrics/pull/4#issuecomment-224951827","issue_url":"https://api.github.com/repos/spotify/semantic-metrics/issues/4","id":224951827,"user":{"login":"udoprog","id":111092,"avatar_url":"https://avatars.githubusercontent.com/u/111092?v=3","gravatar_id":"","url":"https://api.github.com/users/udoprog","html_url":"https://github.com/udoprog","followers_url":"https://api.github.com/users/udoprog/followers","following_url":"https://api.github.com/users/udoprog/following{/other_user}","gists_url":"https://api.github.com/users/udoprog/gists{/gist_id}","starred_url":"https://api.github.com/users/udoprog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/udoprog/subscriptions","organizations_url":"https://api.github.com/users/udoprog/orgs","repos_url":"https://api.github.com/users/udoprog/repos","events_url":"https://api.github.com/users/udoprog/events{/privacy}","received_events_url":"https://api.github.com/users/udoprog/received_events","type":"User","site_admin":false},"created_at":"2016-06-09T16:34:46Z","updated_at":"2016-06-09T16:34:46Z","body":"It seems like the `RemoteMeter` and `RemoteSemanticMetricRegistry` implementation is pretty much tied to semantic-aggregator.\r\n\r\nDue to this, I think it would make sense to have `RemoteSemanticMetricRegistry` being an interface in the API package, and and implementation named `SemanticAggregatorMetricRegistry` could be given its own module outside of core (remote-semantic-aggregator). This way the core dependencies can be slimmed as well."}}
{"actor":{"display_login":"udoprog"}}