[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-16 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/2745#discussion_r18998585 --- Diff: docs/configuration.md --- @@ -962,7 +954,7 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-16 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-59457546 Ok, I merge this. Thanks. --- 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-3890][Docs]remove redundant spark.execu...

2014-10-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2745 --- 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-3890][Docs]remove redundant spark.execu...

2014-10-13 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2745#discussion_r18776416 --- Diff: docs/configuration.md --- @@ -959,10 +950,10 @@ Apart from these, the following properties are also available, and may be useful standard a

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-13 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2745#discussion_r18776443 --- Diff: docs/configuration.md --- @@ -880,8 +872,7 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-13 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-58916414 @srowen I checked and found the line contains more than 100 charactors already, so keep the wrapping. The period is also deleted. --- If your project is set up

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-58771859 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21652/consoleFull) for PR 2745 at commit

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-58772831 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21652/consoleFull) for PR 2745 at commit

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

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

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-10 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-58687961 It could be because it belongs to both categories (run time environment and application behavior), but yes I think we should remove one. --- If your project is set

[GitHub] spark pull request: [SPARK-3890][Docs]remove redundant spark.execu...

2014-10-10 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the pull request: https://github.com/apache/spark/pull/2745#issuecomment-58735051 Actually I don't understant what `Runtime Environment` category means. --- If your project is set up for it, you can reply to this email and have your reply