{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/7629","labels_url":"https://api.github.com/repos/apache/spark/issues/7629/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/7629/comments","events_url":"https://api.github.com/repos/apache/spark/issues/7629/events","html_url":"https://github.com/apache/spark/pull/7629","id":96926215,"number":7629,"title":"[SPARK-9284] [tests] Allow all tests to run without an assembly.","user":{"login":"vanzin","id":1694083,"avatar_url":"https://avatars.githubusercontent.com/u/1694083?v=3","gravatar_id":"","url":"https://api.github.com/users/vanzin","html_url":"https://github.com/vanzin","followers_url":"https://api.github.com/users/vanzin/followers","following_url":"https://api.github.com/users/vanzin/following{/other_user}","gists_url":"https://api.github.com/users/vanzin/gists{/gist_id}","starred_url":"https://api.github.com/users/vanzin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vanzin/subscriptions","organizations_url":"https://api.github.com/users/vanzin/orgs","repos_url":"https://api.github.com/users/vanzin/repos","events_url":"https://api.github.com/users/vanzin/events{/privacy}","received_events_url":"https://api.github.com/users/vanzin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":43,"created_at":"2015-07-23T23:31:32Z","updated_at":"2015-07-28T22:41:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/7629","html_url":"https://github.com/apache/spark/pull/7629","diff_url":"https://github.com/apache/spark/pull/7629.diff","patch_url":"https://github.com/apache/spark/pull/7629.patch"},"body":"This change aims at speeding up the dev cycle a little bit, by making\r\nsure that all tests behave the same w.r.t. where the code to be tested\r\nis loaded from. Namely, that means that tests don't rely on the assembly\r\nanymore, rather loading all needed classes from the build directories.\r\n\r\nThe main change is to make sure all build directories (classes and test-classes)\r\nare added to the classpath of child processes when running tests.\r\n\r\nYarnClusterSuite required some custom code since the executors are run\r\ndifferently (i.e. not through the launcher library, like standalone and\r\nMesos do).\r\n\r\nI also found a couple of tests that could leak a SparkContext on failure,\r\nand added code to handle those.\r\n\r\nWith this patch, it's possible to run the following command from a clean\r\nsource directory and have all tests pass:\r\n\r\n  mvn -Pyarn -Phadoop-2.4 -Phive-thriftserver install\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/125773056","html_url":"https://github.com/apache/spark/pull/7629#issuecomment-125773056","issue_url":"https://api.github.com/repos/apache/spark/issues/7629","id":125773056,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2015-07-28T22:41:17Z","updated_at":"2015-07-28T22:41:17Z","body":" Merged build triggered."}}