{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/9175","labels_url":"https://api.github.com/repos/apache/spark/issues/9175/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/9175/comments","events_url":"https://api.github.com/repos/apache/spark/issues/9175/events","html_url":"https://github.com/apache/spark/pull/9175","id":112311880,"number":9175,"title":"Minor cleanup of ShuffleMapStage.outputLocs code.","user":{"login":"rxin","id":323388,"avatar_url":"https://avatars.githubusercontent.com/u/323388?v=3","gravatar_id":"","url":"https://api.github.com/users/rxin","html_url":"https://github.com/rxin","followers_url":"https://api.github.com/users/rxin/followers","following_url":"https://api.github.com/users/rxin/following{/other_user}","gists_url":"https://api.github.com/users/rxin/gists{/gist_id}","starred_url":"https://api.github.com/users/rxin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rxin/subscriptions","organizations_url":"https://api.github.com/users/rxin/orgs","repos_url":"https://api.github.com/users/rxin/repos","events_url":"https://api.github.com/users/rxin/events{/privacy}","received_events_url":"https://api.github.com/users/rxin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-10-20T08:19:53Z","updated_at":"2015-10-21T20:24:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/9175","html_url":"https://github.com/apache/spark/pull/9175","diff_url":"https://github.com/apache/spark/pull/9175.diff","patch_url":"https://github.com/apache/spark/pull/9175.patch"},"body":"I was looking at this code and found the documentation to be insufficient. I added more documentation, and refactored some relevant code path slightly to improve encapsulation. There are more that I want to do, but I want to get these changes in before doing more work.\r\n\r\nMy goal is to reduce exposing internal fields directly in ShuffleMapStage to improve encapsulation. After this change, DAGScheduler no longer directly writes outputLocs. There are still 3 places that reads outputLocs directly, but we can change those later.\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/150014390","html_url":"https://github.com/apache/spark/pull/9175#issuecomment-150014390","issue_url":"https://api.github.com/repos/apache/spark/issues/9175","id":150014390,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars.githubusercontent.com/u/8108735?v=3","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2015-10-21T20:24:43Z","updated_at":"2015-10-21T20:24:43Z","body":"**[Test build #44088 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44088/consoleFull)** for PR 9175 at commit [`59319e1`](https://github.com/apache/spark/commit/59319e10be8d8e89ab3872162275bdd94a357799)."}}