{"action":"created","issue":{"url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/440","labels_url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/440/labels{/name}","comments_url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/440/comments","events_url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/440/events","html_url":"https://github.com/GoogleCloudPlatform/heapster/pull/440","id":98055723,"number":440,"title":"[Model 10] Derived stats implementation and APIs","user":{"login":"afein","id":680441,"avatar_url":"https://avatars.githubusercontent.com/u/680441?v=3","gravatar_id":"","url":"https://api.github.com/users/afein","html_url":"https://github.com/afein","followers_url":"https://api.github.com/users/afein/followers","following_url":"https://api.github.com/users/afein/following{/other_user}","gists_url":"https://api.github.com/users/afein/gists{/gist_id}","starred_url":"https://api.github.com/users/afein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afein/subscriptions","organizations_url":"https://api.github.com/users/afein/orgs","repos_url":"https://api.github.com/users/afein/repos","events_url":"https://api.github.com/users/afein/events{/privacy}","received_events_url":"https://api.github.com/users/afein/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-07-30T00:16:13Z","updated_at":"2015-08-06T23:11:52Z","closed_at":"2015-08-06T23:11:52Z","pull_request":{"url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/pulls/440","html_url":"https://github.com/GoogleCloudPlatform/heapster/pull/440","diff_url":"https://github.com/GoogleCloudPlatform/heapster/pull/440.diff","patch_url":"https://github.com/GoogleCloudPlatform/heapster/pull/440.patch"},"body":"This PR includes the initial implementation of derived stats on top of the existing model.\r\nThis implementation is highly redundant in memory, as DayStores are used for all measured metrics, including limits. \r\n\r\nThe future plan is to only allow derived stats calculations when these are explicitly enabled for specific stats (incoming in [Model 12], most probably).\r\n\r\nTest coverage of the model is at 96.9%.\r\n\r\nAreas that can be structurally and functionally improved:\r\n- DayStore and StatStore implementation\r\n\r\n\\cc @rjnagal @vishh @mvdan "},"comment":{"url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/comments/128536401","html_url":"https://github.com/GoogleCloudPlatform/heapster/pull/440#issuecomment-128536401","issue_url":"https://api.github.com/repos/GoogleCloudPlatform/heapster/issues/440","id":128536401,"user":{"login":"afein","id":680441,"avatar_url":"https://avatars.githubusercontent.com/u/680441?v=3","gravatar_id":"","url":"https://api.github.com/users/afein","html_url":"https://github.com/afein","followers_url":"https://api.github.com/users/afein/followers","following_url":"https://api.github.com/users/afein/following{/other_user}","gists_url":"https://api.github.com/users/afein/gists{/gist_id}","starred_url":"https://api.github.com/users/afein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afein/subscriptions","organizations_url":"https://api.github.com/users/afein/orgs","repos_url":"https://api.github.com/users/afein/repos","events_url":"https://api.github.com/users/afein/events{/privacy}","received_events_url":"https://api.github.com/users/afein/received_events","type":"User","site_admin":false},"created_at":"2015-08-06T23:11:52Z","updated_at":"2015-08-06T23:11:52Z","body":"closing this PR to split it into multiple ones and reimplement the StatStore."}}