{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/9127","labels_url":"https://api.github.com/repos/apache/spark/issues/9127/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/9127/comments","events_url":"https://api.github.com/repos/apache/spark/issues/9127/events","html_url":"https://github.com/apache/spark/pull/9127","id":111518656,"number":9127,"title":"[SPARK-10984] [WIP] Simplify *MemoryManager class structure","user":{"login":"JoshRosen","id":50748,"avatar_url":"https://avatars.githubusercontent.com/u/50748?v=3","gravatar_id":"","url":"https://api.github.com/users/JoshRosen","html_url":"https://github.com/JoshRosen","followers_url":"https://api.github.com/users/JoshRosen/followers","following_url":"https://api.github.com/users/JoshRosen/following{/other_user}","gists_url":"https://api.github.com/users/JoshRosen/gists{/gist_id}","starred_url":"https://api.github.com/users/JoshRosen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JoshRosen/subscriptions","organizations_url":"https://api.github.com/users/JoshRosen/orgs","repos_url":"https://api.github.com/users/JoshRosen/repos","events_url":"https://api.github.com/users/JoshRosen/events{/privacy}","received_events_url":"https://api.github.com/users/JoshRosen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-15T00:02:44Z","updated_at":"2015-10-15T16:12:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/9127","html_url":"https://github.com/apache/spark/pull/9127","diff_url":"https://github.com/apache/spark/pull/9127.diff","patch_url":"https://github.com/apache/spark/pull/9127.patch"},"body":"This patch refactors the MemoryManager class structure. After #9000, Spark had the following classes:\r\n\r\n- MemoryManager\r\n- StaticMemoryManager\r\n- ExecutorMemoryManager\r\n- TaskMemoryManager\r\n- ShuffleMemoryManager\r\n\r\nThis is fairly confusing. To simplify things, this patch consolidates several of these classes:\r\n\r\n- ShuffleMemoryManager and ExecutorMemoryManager were merged into MemoryManager.\r\n- TaskMemoryManager is moved into Spark Core.\r\n\r\nThis patch is currently a work-in-progress; the remaining TODOs:\r\n\r\n- [x] Merge ExecutorMemoryManager into MemoryManager.\r\n- [ ] Merge ShuffleMemoryManager into MemoryManager.\r\n- [x] Move TaskMemoryManager from `spark-unsafe` to `spark-core`.\r\n- [ ] Refactor TaskMemoryManager interactions so tasks use only this and not both this and ShuffleMemoryManager.\r\n- [ ] Misc. docs and cleanup."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/148440578","html_url":"https://github.com/apache/spark/pull/9127#issuecomment-148440578","issue_url":"https://api.github.com/repos/apache/spark/issues/9127","id":148440578,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2015-10-15T16:12:21Z","updated_at":"2015-10-15T16:12:21Z","body":"Merged build started."}}