[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread gzm55
Github user gzm55 commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10462859 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,7 +381,17 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10478785 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,7 +381,17 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10498244 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37372654 Merged build started. --- 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-1064

2014-03-11 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10505297 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,9 +381,48 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37374812 Merged build finished. --- 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-1064

2014-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37374814 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13121/ --- If your project

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10506184 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,9 +381,48 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37377235 Looks good thanks @sryza and to @gzm55 for pointing out the issue. @sryza y u keep changing API's? I'll merge this. --- If your project is set up for it, you

[GitHub] spark pull request: SPARK-1064

2014-03-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/102 --- 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-1064

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37156373 Merged build finished. --- 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-1064

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37156375 One or more automated tests failed Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13086/ --- If your

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10457908 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10458176 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,7 +381,17 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10458859 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10458903 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,7 +381,17 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10459510 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10459709 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -379,7 +381,17 @@ object ClientBase { // Based on code

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10459998 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-10 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/102#discussion_r10460757 --- Diff: project/SparkBuild.scala --- @@ -236,7 +236,8 @@ object SparkBuild extends Build { com.novocode % junit-interface % 0.10 % test,

[GitHub] spark pull request: SPARK-1064

2014-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37154500 Merged build started. --- 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-1064

2014-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37154499 Merged build triggered. --- 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

[GitHub] spark pull request: SPARK-1064

2014-03-07 Thread sryza
GitHub user sryza opened a pull request: https://github.com/apache/spark/pull/102 SPARK-1064 This reopens PR 649 from incubator-spark against the new repo You can merge this pull request into a Git repository by running: $ git pull https://github.com/sryza/spark

[GitHub] spark pull request: SPARK-1064

2014-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/102#issuecomment-37090694 Merged build finished. --- 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