{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/9066","labels_url":"https://api.github.com/repos/apache/spark/issues/9066/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/9066/comments","events_url":"https://api.github.com/repos/apache/spark/issues/9066/events","html_url":"https://github.com/apache/spark/pull/9066","id":110832770,"number":9066,"title":"[SPARK-11053] Remove use of KVIterator in SortBasedAggregationIterator","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-11T04:30:13Z","updated_at":"2015-10-11T06:05:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/9066","html_url":"https://github.com/apache/spark/pull/9066","diff_url":"https://github.com/apache/spark/pull/9066.diff","patch_url":"https://github.com/apache/spark/pull/9066.patch"},"body":"SortBasedAggregationIterator uses a KVIterator interface in order to process input rows as key-value pairs, but this use of KVIterator is unnecessary, slightly complicates the code, and might hurt performance. This patch refactors this code to remove the use of this extra layer of iterator wrapping and simplifies other parts of the code in the process."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/147161394","html_url":"https://github.com/apache/spark/pull/9066#issuecomment-147161394","issue_url":"https://api.github.com/repos/apache/spark/issues/9066","id":147161394,"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},"created_at":"2015-10-11T06:05:00Z","updated_at":"2015-10-11T06:05:00Z","body":"Looks ok to me. I will leave it to @yhuai for the detailed review."}}