{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-beam/issues/256","repository_url":"https://api.github.com/repos/apache/incubator-beam","labels_url":"https://api.github.com/repos/apache/incubator-beam/issues/256/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-beam/issues/256/comments","events_url":"https://api.github.com/repos/apache/incubator-beam/issues/256/events","html_url":"https://github.com/apache/incubator-beam/pull/256","id":151507256,"number":256,"title":"[BEAM-77] Move InProcessRunner to its own module","user":{"login":"kennknowles","id":575353,"avatar_url":"https://avatars.githubusercontent.com/u/575353?v=3","gravatar_id":"","url":"https://api.github.com/users/kennknowles","html_url":"https://github.com/kennknowles","followers_url":"https://api.github.com/users/kennknowles/followers","following_url":"https://api.github.com/users/kennknowles/following{/other_user}","gists_url":"https://api.github.com/users/kennknowles/gists{/gist_id}","starred_url":"https://api.github.com/users/kennknowles/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennknowles/subscriptions","organizations_url":"https://api.github.com/users/kennknowles/orgs","repos_url":"https://api.github.com/users/kennknowles/repos","events_url":"https://api.github.com/users/kennknowles/events{/privacy}","received_events_url":"https://api.github.com/users/kennknowles/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-04-27T22:32:53Z","updated_at":"2016-04-29T15:34:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-beam/pulls/256","html_url":"https://github.com/apache/incubator-beam/pull/256","diff_url":"https://github.com/apache/incubator-beam/pull/256.diff","patch_url":"https://github.com/apache/incubator-beam/pull/256.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\nI believe since the `DirectPipelineRunner` is still in place, this safe to do immediately. Then we can begin to run all tests on this runner in the same way we do for all runners, gaining coverage of both. Finally, we can simply delete `DirectPipelineRunner` which will expose any tests that require the `NeedsRunner` category."},"comment":{"url":"https://api.github.com/repos/apache/incubator-beam/issues/comments/215761248","html_url":"https://github.com/apache/incubator-beam/pull/256#issuecomment-215761248","issue_url":"https://api.github.com/repos/apache/incubator-beam/issues/256","id":215761248,"user":{"login":"kennknowles","id":575353,"avatar_url":"https://avatars.githubusercontent.com/u/575353?v=3","gravatar_id":"","url":"https://api.github.com/users/kennknowles","html_url":"https://github.com/kennknowles","followers_url":"https://api.github.com/users/kennknowles/followers","following_url":"https://api.github.com/users/kennknowles/following{/other_user}","gists_url":"https://api.github.com/users/kennknowles/gists{/gist_id}","starred_url":"https://api.github.com/users/kennknowles/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennknowles/subscriptions","organizations_url":"https://api.github.com/users/kennknowles/orgs","repos_url":"https://api.github.com/users/kennknowles/repos","events_url":"https://api.github.com/users/kennknowles/events{/privacy}","received_events_url":"https://api.github.com/users/kennknowles/received_events","type":"User","site_admin":false},"created_at":"2016-04-29T15:34:40Z","updated_at":"2016-04-29T15:34:40Z","body":"I believe Travis-CI failures are memory/performance issues in `InProcessPipelineRunner` not related to this PR, since there are no code changes here."}}