{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/5694","labels_url":"https://api.github.com/repos/apache/spark/issues/5694/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/5694/comments","events_url":"https://api.github.com/repos/apache/spark/issues/5694/events","html_url":"https://github.com/apache/spark/pull/5694","id":70800338,"number":5694,"title":"[SPARK-7017][Build][Project Infra]: Refactor dev/run-tests into Python","user":{"login":"brennonyork","id":2230193,"avatar_url":"https://avatars.githubusercontent.com/u/2230193?v=3","gravatar_id":"","url":"https://api.github.com/users/brennonyork","html_url":"https://github.com/brennonyork","followers_url":"https://api.github.com/users/brennonyork/followers","following_url":"https://api.github.com/users/brennonyork/following{/other_user}","gists_url":"https://api.github.com/users/brennonyork/gists{/gist_id}","starred_url":"https://api.github.com/users/brennonyork/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brennonyork/subscriptions","organizations_url":"https://api.github.com/users/brennonyork/orgs","repos_url":"https://api.github.com/users/brennonyork/repos","events_url":"https://api.github.com/users/brennonyork/events{/privacy}","received_events_url":"https://api.github.com/users/brennonyork/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":135,"created_at":"2015-04-24T21:35:35Z","updated_at":"2015-05-13T16:07:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/5694","html_url":"https://github.com/apache/spark/pull/5694","diff_url":"https://github.com/apache/spark/pull/5694.diff","patch_url":"https://github.com/apache/spark/pull/5694.patch"},"body":"All, this is a first attempt at refactoring `dev/run-tests` into Python. Initially I merely converted all Bash calls over to Python, then moved to a much more modular approach (more functions, moved the calls around, etc.). What is here is the initial culmination and should provide a great base to various downstream issues (e.g. SPARK-7016, modularize / parallelize testing, etc.). Would love comments / suggestions for this initial first step!\r\n\r\n/cc @srowen @pwendell @nchammas"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/101726519","html_url":"https://github.com/apache/spark/pull/5694#issuecomment-101726519","issue_url":"https://api.github.com/repos/apache/spark/issues/5694","id":101726519,"user":{"login":"nchammas","id":1039369,"avatar_url":"https://avatars.githubusercontent.com/u/1039369?v=3","gravatar_id":"","url":"https://api.github.com/users/nchammas","html_url":"https://github.com/nchammas","followers_url":"https://api.github.com/users/nchammas/followers","following_url":"https://api.github.com/users/nchammas/following{/other_user}","gists_url":"https://api.github.com/users/nchammas/gists{/gist_id}","starred_url":"https://api.github.com/users/nchammas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nchammas/subscriptions","organizations_url":"https://api.github.com/users/nchammas/orgs","repos_url":"https://api.github.com/users/nchammas/repos","events_url":"https://api.github.com/users/nchammas/events{/privacy}","received_events_url":"https://api.github.com/users/nchammas/received_events","type":"User","site_admin":false},"created_at":"2015-05-13T16:07:45Z","updated_at":"2015-05-13T16:07:45Z","body":"> could I get a few eyes on this one?\r\n\r\nDoesn't look like you've made any big changes since my last review, so I can say this patch LGTM overall.\r\n\r\nYou more recent commit mentions [SPARK-6908](https://issues.apache.org/jira/browse/SPARK-6908); I'm guessing you pulled those changes [from here](https://github.com/apache/spark/pull/5876/files#diff-3dc5ad3a9953df8025880bf443153d6e).\r\n\r\nYou might also want to incorporate the changes proposed in #5955 by @yhuai."}}