{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-beam/issues/1251","repository_url":"https://api.github.com/repos/apache/incubator-beam","labels_url":"https://api.github.com/repos/apache/incubator-beam/issues/1251/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-beam/issues/1251/comments","events_url":"https://api.github.com/repos/apache/incubator-beam/issues/1251/events","html_url":"https://github.com/apache/incubator-beam/pull/1251","id":186657741,"number":1251,"title":"[BEAM-788] Implement GroupAlsoByWindowsEvaluator Directly","user":{"login":"tgroh","id":429866,"avatar_url":"https://avatars.githubusercontent.com/u/429866?v=3","gravatar_id":"","url":"https://api.github.com/users/tgroh","html_url":"https://github.com/tgroh","followers_url":"https://api.github.com/users/tgroh/followers","following_url":"https://api.github.com/users/tgroh/following{/other_user}","gists_url":"https://api.github.com/users/tgroh/gists{/gist_id}","starred_url":"https://api.github.com/users/tgroh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tgroh/subscriptions","organizations_url":"https://api.github.com/users/tgroh/orgs","repos_url":"https://api.github.com/users/tgroh/repos","events_url":"https://api.github.com/users/tgroh/events{/privacy}","received_events_url":"https://api.github.com/users/tgroh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-01T21:50:44Z","updated_at":"2016-11-02T07:42:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-beam/pulls/1251","html_url":"https://github.com/apache/incubator-beam/pull/1251","diff_url":"https://github.com/apache/incubator-beam/pull/1251.diff","patch_url":"https://github.com/apache/incubator-beam/pull/1251.patch"},"body":"Be sure to do all of the following to help us incorporate your contribution\r\nquickly and easily:\r\n\r\n - [x] Make sure the PR title is formatted like:\r\n   `[BEAM-<Jira issue #>] Description of pull request`\r\n - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable\r\n       Travis-CI on your fork and ensure the whole test matrix passes).\r\n - [x] Replace `<Jira issue #>` in the title with the actual Jira issue\r\n       number, if there is one.\r\n - [x] If this contribution is large, please file an Apache\r\n       [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.txt).\r\n\r\n---\r\n\r\nInvoke the ReduceFnRunner methods directly rather than by constructing a\r\nGroupAlsoByWindowDoFn and a ParDoEvaluator. Instead, construct a\r\nReduceFnRunner, ReduceFn, and invoke the methods directly."},"comment":{"url":"https://api.github.com/repos/apache/incubator-beam/issues/comments/257793746","html_url":"https://github.com/apache/incubator-beam/pull/1251#issuecomment-257793746","issue_url":"https://api.github.com/repos/apache/incubator-beam/issues/1251","id":257793746,"user":{"login":"amitsela","id":4245753,"avatar_url":"https://avatars.githubusercontent.com/u/4245753?v=3","gravatar_id":"","url":"https://api.github.com/users/amitsela","html_url":"https://github.com/amitsela","followers_url":"https://api.github.com/users/amitsela/followers","following_url":"https://api.github.com/users/amitsela/following{/other_user}","gists_url":"https://api.github.com/users/amitsela/gists{/gist_id}","starred_url":"https://api.github.com/users/amitsela/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amitsela/subscriptions","organizations_url":"https://api.github.com/users/amitsela/orgs","repos_url":"https://api.github.com/users/amitsela/repos","events_url":"https://api.github.com/users/amitsela/events{/privacy}","received_events_url":"https://api.github.com/users/amitsela/received_events","type":"User","site_admin":false},"created_at":"2016-11-02T07:42:43Z","updated_at":"2016-11-02T07:42:43Z","body":"@tgroh shouldn't this be at the `runner-core` level, instead of having each runner implement this ?"}}
{"actor":{"display_login":"amitsela"}}