[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-28 Thread sryza
Github user sryza commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36368855 Thanks Tom! --- 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-1032. If Yarn app fails before registeri...

2014-02-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/28 --- 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 enable

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-28 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36362791 I committed this. Thanks Sandy! --- 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 h

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36255394 I personally like the way 4 spaces looks too. The style guide isn't clear on what its supposed to be. I'll assume it falls under the 4 space rule similar to functions:

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36226280 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 t

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36226282 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12915/ --- If your project i

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36223986 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 th

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36223996 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 have

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/28#issuecomment-36223982 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 have

[GitHub] spark pull request: SPARK-1032. If Yarn app fails before registeri...

2014-02-27 Thread sryza
GitHub user sryza opened a pull request: https://github.com/apache/spark/pull/28 SPARK-1032. If Yarn app fails before registering, app master stays aroun... ...d long after This reopens https://github.com/apache/incubator-spark/pull/648 against the new repo. You can merge