[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-24 Thread chesterxgchen
Github user chesterxgchen closed the pull request at: https://github.com/apache/spark/pull/2090 --- 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

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-24 Thread chesterxgchen
Github user chesterxgchen commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53211490 Patrick The release version is incorrect in the YARN pom intentionally (it's a weird artifact of the way we publish builds). It would nice that someone

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-24 Thread chesterxgchen
Github user chesterxgchen commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53212454 BTW, I have to change the pom.xml in order to fix the unit test --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53143937 QA results for PR 2090:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/2090#discussion_r16608088 --- Diff: yarn/common/src/test/scala/org/apache/spark/deploy/yarn/ClientBaseSuite.scala --- @@ -199,10 +199,7 @@ class ClientBaseSuite extends FunSuite with

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53088236 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-3177 SPARK-3175

2014-08-22 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53137965 Jenkins, test this please. Thanks LGTM pending tests. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53142551 Jenkins, test this please. --- 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

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53142581 Wait actually - the current change made to the unit test causes the unit test to be entirely useless (it just compares the output of a function to itself. --- If your

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53142611 QA tests have started for PR 2090. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19093/consoleFull ---

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-22 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53142606 The release version is incorrect in the YARN pom intentionally (it's a weird artifact of the way we publish builds). If you want to build branch-1.1 please checkout the

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-21 Thread chesterxgchen
GitHub user chesterxgchen opened a pull request: https://github.com/apache/spark/pull/2090 Spark-3177 SPARK-3175 SPARK-3177 : Yarn-alpha ClientBaseSuite Unit test failed SPARK-3175 : Branch-1.1 SBT build failed for Yarn-Alpha You can merge this pull request into a Git

[GitHub] spark pull request: Spark-3177 SPARK-3175

2014-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2090#issuecomment-53016829 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