[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-05 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145525454 @srowen , what happened here, why was this closed, it looks like it was committed but I don't see anyone +1. I was mostly good with it but it would have been nice to

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145527571 Ah, I thought you had reviewed this and the previous iteration of this PR also? seemed pretty cut and dried to me at this stage. I don't know how long to wait for your

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8968 --- 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-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/8968 [SPARK-9570] [DOCS] Consistent recommendation for submitting spark apps to YARN, -master yarn --deploy-mode x vs -master yarn-x'. Recommend `--master yarn --deploy-mode {cluster,client}`

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145146090 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

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145146077 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-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145147435 [Test build #43196 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43196/consoleFull) for PR 8968 at commit

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread nssalian
Github user nssalian commented on a diff in the pull request: https://github.com/apache/spark/pull/8968#discussion_r41064621 --- Diff: docs/submitting-applications.md --- @@ -122,21 +123,23 @@ The master URL passed to Spark can be in one of the following formats:

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/8968#discussion_r41064265 --- Diff: docs/running-on-yarn.md --- @@ -388,6 +390,6 @@ If you need a reference to the proper location to put log files in the YARN so t # Important

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/8968#discussion_r41064442 --- Diff: docs/submitting-applications.md --- @@ -122,21 +123,23 @@ The master URL passed to Spark can be in one of the following formats:

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145151567 [Test build #43196 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43196/console) for PR 8968 at commit

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145151605 Merged build finished. Test FAILed. --- 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-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8968#discussion_r41068987 --- Diff: docs/submitting-applications.md --- @@ -122,21 +123,23 @@ The master URL passed to Spark can be in one of the following formats:

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145159238 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-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145159254 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

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145160361 [Test build #43199 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43199/consoleFull) for PR 8968 at commit

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145151608 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145186625 Merged build finished. Test PASSed. --- 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-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145186579 [Test build #43199 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43199/console) for PR 8968 at commit

[GitHub] spark pull request: [SPARK-9570] [DOCS] Consistent recommendation ...

2015-10-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8968#issuecomment-145186626 Test PASSed. Refer to this link for build results (access rights to CI server needed):