[GitHub] spark pull request #22922: [SPARK-25909] fix documentation on cluster manage...

2018-11-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/22922 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #22922: [SPARK-25909] fix documentation on cluster manage...

2018-11-01 Thread jameslamb
Github user jameslamb commented on a diff in the pull request: https://github.com/apache/spark/pull/22922#discussion_r230163716 --- Diff: docs/cluster-overview.md --- @@ -45,7 +45,7 @@ There are several useful things to note about this architecture: # Cluster Manager

[GitHub] spark pull request #22922: [SPARK-25909] fix documentation on cluster manage...

2018-11-01 Thread jameslamb
Github user jameslamb commented on a diff in the pull request: https://github.com/apache/spark/pull/22922#discussion_r230163184 --- Diff: docs/cluster-overview.md --- @@ -45,7 +45,7 @@ There are several useful things to note about this architecture: # Cluster Manager

[GitHub] spark pull request #22922: [SPARK-25909] fix documentation on cluster manage...

2018-11-01 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22922#discussion_r230159527 --- Diff: docs/cluster-overview.md --- @@ -45,7 +45,7 @@ There are several useful things to note about this architecture: # Cluster Manager Types

[GitHub] spark pull request #22922: [SPARK-25909] fix documentation on cluster manage...

2018-11-01 Thread jameslamb
GitHub user jameslamb opened a pull request: https://github.com/apache/spark/pull/22922 [SPARK-25909] fix documentation on cluster managers ## What changes were proposed in this pull request? Propose changing the documentation to state that there are 4, not 3, cluster