{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15252","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15252/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15252/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15252/events","html_url":"https://github.com/apache/spark/pull/15252","id":179390548,"number":15252,"title":"[SPARK-17677][SQL] Break WindowExec.scala into multiple files","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,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-27T03:41:38Z","updated_at":"2016-09-27T08:02:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15252","html_url":"https://github.com/apache/spark/pull/15252","diff_url":"https://github.com/apache/spark/pull/15252.diff","patch_url":"https://github.com/apache/spark/pull/15252.patch"},"body":"## What changes were proposed in this pull request?\r\nAs of Spark 2.0, all the window function execution code are in WindowExec.scala. This file is pretty large (over 1k loc) and has a lot of different abstractions in them. This patch creates a new package sql.execution.window, moves WindowExec.scala in it, and breaks WindowExec.scala into multiple, more maintainable pieces:\r\n\r\n- AggregateProcessor.scala\r\n- BoundOrdering.scala\r\n- RowBuffer.scala\r\n- WindowExec\r\n- WindowFunctionFrame.scala\r\n\r\n## How was this patch tested?\r\nThis patch mostly moves code around, and should not change any existing test coverage.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/249794177","html_url":"https://github.com/apache/spark/pull/15252#issuecomment-249794177","issue_url":"https://api.github.com/repos/apache/spark/issues/15252","id":249794177,"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":"2016-09-27T08:02:32Z","updated_at":"2016-09-27T08:02:32Z","body":"**[Test build #65956 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65956/consoleFull)** for PR 15252 at commit [`dbe06ad`](https://github.com/apache/spark/commit/dbe06ad97ad6032684cd7807b4df5f1c549c9e03)."}}
{"actor":{"display_login":"SparkQA"}}