{"action":"opened","issue":{"url":"https://api.github.com/repos/census-instrumentation/opencensus-python/issues/388","repository_url":"https://api.github.com/repos/census-instrumentation/opencensus-python","labels_url":"https://api.github.com/repos/census-instrumentation/opencensus-python/issues/388/labels{/name}","comments_url":"https://api.github.com/repos/census-instrumentation/opencensus-python/issues/388/comments","events_url":"https://api.github.com/repos/census-instrumentation/opencensus-python/issues/388/events","html_url":"https://github.com/census-instrumentation/opencensus-python/issues/388","id":378494099,"node_id":"MDU6SXNzdWUzNzg0OTQwOTk=","number":388,"title":"Remove unused DistributionAggregation fields","user":{"login":"c24t","id":43799105,"node_id":"MDQ6VXNlcjQzNzk5MTA1","avatar_url":"https://avatars3.githubusercontent.com/u/43799105?v=4","gravatar_id":"","url":"https://api.github.com/users/c24t","html_url":"https://github.com/c24t","followers_url":"https://api.github.com/users/c24t/followers","following_url":"https://api.github.com/users/c24t/following{/other_user}","gists_url":"https://api.github.com/users/c24t/gists{/gist_id}","starred_url":"https://api.github.com/users/c24t/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/c24t/subscriptions","organizations_url":"https://api.github.com/users/c24t/orgs","repos_url":"https://api.github.com/users/c24t/repos","events_url":"https://api.github.com/users/c24t/events{/privacy}","received_events_url":"https://api.github.com/users/c24t/received_events","type":"User","site_admin":false},"labels":[{"id":1119887019,"node_id":"MDU6TGFiZWwxMTE5ODg3MDE5","url":"https://api.github.com/repos/census-instrumentation/opencensus-python/labels/tech%20debt","name":"tech debt","color":"dbf977","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-07T22:20:39Z","updated_at":"2018-11-07T22:20:39Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"[`DistributionAggregation`](https://github.com/census-instrumentation/opencensus-python/blob/7dd2a44ce0bcda77f902a34a91c590dfcbb4afea/opencensus/stats/aggregation.py#L111-L146)'s `boundaries` and `distribution` attributes are redundant with `DistributionAggregationData`, and unused except for [in the prometheus exporter](https://github.com/census-instrumentation/opencensus-python/blob/7dd2a44ce0bcda77f902a34a91c590dfcbb4afea/opencensus/stats/exporters/prometheus_exporter.py#L176-L181), which looks like it should be using the aggregation data instead.\r\n\r\nMore generally, I don't see any reason to have separate `*Aggregation` and `*DataAggregation` classes. These were added in #149; @kcooperstein, @songy23, @liyanhui1228 am I missing something here?"}}
{"actor":{"display_login":"c24t"}}