{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto/issues/11218","repository_url":"https://api.github.com/repos/prestodb/presto","labels_url":"https://api.github.com/repos/prestodb/presto/issues/11218/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto/issues/11218/comments","events_url":"https://api.github.com/repos/prestodb/presto/issues/11218/events","html_url":"https://github.com/prestodb/presto/pull/11218","id":348523583,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA2ODU1MDYy","number":11218,"title":"Fix memory accounting leak in AggregationOperator","user":{"login":"nezihyigitbasi","id":1223839,"node_id":"MDQ6VXNlcjEyMjM4Mzk=","avatar_url":"https://avatars3.githubusercontent.com/u/1223839?v=4","gravatar_id":"","url":"https://api.github.com/users/nezihyigitbasi","html_url":"https://github.com/nezihyigitbasi","followers_url":"https://api.github.com/users/nezihyigitbasi/followers","following_url":"https://api.github.com/users/nezihyigitbasi/following{/other_user}","gists_url":"https://api.github.com/users/nezihyigitbasi/gists{/gist_id}","starred_url":"https://api.github.com/users/nezihyigitbasi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nezihyigitbasi/subscriptions","organizations_url":"https://api.github.com/users/nezihyigitbasi/orgs","repos_url":"https://api.github.com/users/nezihyigitbasi/repos","events_url":"https://api.github.com/users/nezihyigitbasi/events{/privacy}","received_events_url":"https://api.github.com/users/nezihyigitbasi/received_events","type":"User","site_admin":false},"labels":[{"id":196992743,"node_id":"MDU6TGFiZWwxOTY5OTI3NDM=","url":"https://api.github.com/repos/prestodb/presto/labels/CLA%20Signed","name":"CLA Signed","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-08-07T23:37:18Z","updated_at":"2018-08-08T03:27:19Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/prestodb/presto/pulls/11218","html_url":"https://github.com/prestodb/presto/pull/11218","diff_url":"https://github.com/prestodb/presto/pull/11218.diff","patch_url":"https://github.com/prestodb/presto/pull/11218.patch"},"body":"If an AggregationOperator is used inside a TableWriterOperator\r\nfor stats collection the AggregationOperator will leak, as the driver\r\nthread will only destroy the TableWriterOperator, and operators\r\nrely on getting destroyed by the driver threads to free up their\r\nallocations.\r\n\r\nThis change also closes the memory context of TableWriterOperator\r\nin close(), however that's just for testing as the memory reserved\r\nby that context will be released when the driver thread destroys its\r\noperator context."},"comment":{"url":"https://api.github.com/repos/prestodb/presto/issues/comments/411272554","html_url":"https://github.com/prestodb/presto/pull/11218#issuecomment-411272554","issue_url":"https://api.github.com/repos/prestodb/presto/issues/11218","id":411272554,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMTI3MjU1NA==","user":{"login":"nezihyigitbasi","id":1223839,"node_id":"MDQ6VXNlcjEyMjM4Mzk=","avatar_url":"https://avatars3.githubusercontent.com/u/1223839?v=4","gravatar_id":"","url":"https://api.github.com/users/nezihyigitbasi","html_url":"https://github.com/nezihyigitbasi","followers_url":"https://api.github.com/users/nezihyigitbasi/followers","following_url":"https://api.github.com/users/nezihyigitbasi/following{/other_user}","gists_url":"https://api.github.com/users/nezihyigitbasi/gists{/gist_id}","starred_url":"https://api.github.com/users/nezihyigitbasi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nezihyigitbasi/subscriptions","organizations_url":"https://api.github.com/users/nezihyigitbasi/orgs","repos_url":"https://api.github.com/users/nezihyigitbasi/repos","events_url":"https://api.github.com/users/nezihyigitbasi/events{/privacy}","received_events_url":"https://api.github.com/users/nezihyigitbasi/received_events","type":"User","site_admin":false},"created_at":"2018-08-08T03:27:19Z","updated_at":"2018-08-08T03:27:19Z","author_association":"CONTRIBUTOR","body":"Augmented an existing test to verify the memory is released properly."}}
{"actor":{"display_login":"nezihyigitbasi"}}