[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2016-03-15 Thread abridgett
Github user abridgett commented on a diff in the pull request: https://github.com/apache/spark/pull/10448#discussion_r56180120 --- Diff: make-distribution.sh --- @@ -159,7 +159,7 @@ fi # Build uber fat JAR cd "$SPARK_HOME" -export MAVEN_OPTS="-Xmx2g

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2016-03-15 Thread nchammas
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/10448#discussion_r56179522 --- Diff: make-distribution.sh --- @@ -159,7 +159,7 @@ fi # Build uber fat JAR cd "$SPARK_HOME" -export MAVEN_OPTS="-Xmx2g

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread abridgett
GitHub user abridgett opened a pull request: https://github.com/apache/spark/pull/10448 [SPARK-12499] [build] don't force MAVEN_OPTS allow the user to override MAVEN_OPTS (2GB wasn't sufficient for me) You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10448#issuecomment-166931510 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10448#issuecomment-166958625 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10448#issuecomment-166959071 **[Test build #2252 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2252/consoleFull)** for PR 10448 at commit

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10448 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10448#issuecomment-167011370 I've merged this into `master` and `branch-1.6`. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-12499] [build] don't force MAVEN_OPTS

2015-12-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10448#issuecomment-166977795 **[Test build #2252 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2252/consoleFull)** for PR 10448 at commit