{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/7219","labels_url":"https://api.github.com/repos/apache/spark/issues/7219/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/7219/comments","events_url":"https://api.github.com/repos/apache/spark/issues/7219/events","html_url":"https://github.com/apache/spark/pull/7219","id":92970534,"number":7219,"title":"[SPARK-8819] Fix build for maven 3.3.x","user":{"login":"andrewor14","id":2133137,"avatar_url":"https://avatars.githubusercontent.com/u/2133137?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewor14","html_url":"https://github.com/andrewor14","followers_url":"https://api.github.com/users/andrewor14/followers","following_url":"https://api.github.com/users/andrewor14/following{/other_user}","gists_url":"https://api.github.com/users/andrewor14/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewor14/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewor14/subscriptions","organizations_url":"https://api.github.com/users/andrewor14/orgs","repos_url":"https://api.github.com/users/andrewor14/repos","events_url":"https://api.github.com/users/andrewor14/events{/privacy}","received_events_url":"https://api.github.com/users/andrewor14/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":63,"created_at":"2015-07-04T04:39:21Z","updated_at":"2015-07-07T08:02:55Z","closed_at":"2015-07-07T02:23:38Z","pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/7219","html_url":"https://github.com/apache/spark/pull/7219","diff_url":"https://github.com/apache/spark/pull/7219.diff","patch_url":"https://github.com/apache/spark/pull/7219.patch"},"body":"This is a workaround for MSHADE-148, which leads to an infinite loop when building Spark with maven 3.3.x. This was originally caused by #6441, which added a bunch of test dependencies on the spark-core test module. Recently, it was revealed by #7193.\r\n\r\nThis patch adds a `-Prelease` profile. If present, it will set `createDependencyReducedPom` to true. The consequences are:\r\n- If you are releasing Spark with this profile, you are fine as long as you use maven 3.2.x or before.\r\n- If you are releasing Spark without this profile, you will run into SPARK-8781.\r\n- If you are not releasing Spark but you are using this profile, you may run into SPARK-8819.\r\n- If you are not releasing Spark and you did not include this profile, you are fine.\r\n\r\nThis is all documented in `pom.xml` and tested locally with both versions of maven."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/119111857","html_url":"https://github.com/apache/spark/pull/7219#issuecomment-119111857","issue_url":"https://api.github.com/repos/apache/spark/issues/7219","id":119111857,"user":{"login":"srowen","id":822522,"avatar_url":"https://avatars.githubusercontent.com/u/822522?v=3","gravatar_id":"","url":"https://api.github.com/users/srowen","html_url":"https://github.com/srowen","followers_url":"https://api.github.com/users/srowen/followers","following_url":"https://api.github.com/users/srowen/following{/other_user}","gists_url":"https://api.github.com/users/srowen/gists{/gist_id}","starred_url":"https://api.github.com/users/srowen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srowen/subscriptions","organizations_url":"https://api.github.com/users/srowen/orgs","repos_url":"https://api.github.com/users/srowen/repos","events_url":"https://api.github.com/users/srowen/events{/privacy}","received_events_url":"https://api.github.com/users/srowen/received_events","type":"User","site_admin":false},"created_at":"2015-07-07T08:02:55Z","updated_at":"2015-07-07T08:02:55Z","body":"Agree that a separate module is a logically nice fix, but probably does have the circular dependency problem and introduce a whole new module for a few lines of code.\r\n\r\nI also saw @pwendell 's message about `release-profile` having side-effects. Darn. I had hoped side effects of setting up like Maven does for release are helpful, like building javadoc, but I guess the javadoc doesn't build on its own. LGTM"}}