You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by andrewor14 <gi...@git.apache.org> on 2014/12/05 23:48:43 UTC

[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

GitHub user andrewor14 opened a pull request:

    https://github.com/apache/spark/pull/3627

    [SPARK-4771][Docs] Document standalone cluster supervise mode

    @tdas looks like streaming already refers to the supervise mode. The link from there is broken though.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/andrewor14/spark document-supervise

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/3627.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3627
    
----
commit 2b55ed22848b32c3bbdf39ec0091f621351cdb22
Author: Andrew Or <an...@databricks.com>
Date:   2014-12-05T22:47:43Z

    Document standalone cluster supervise mode

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21563699
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application on failure. To use
    +this feature, you may pass in the `--supervise` flag to `spark-submit` when launching your
    +application. Then, if you wish to kill an application that is failing repeatedly, you may do so
    --- End diff --
    
    Okay, thats fine then.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21562819
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application on failure. To use
    +this feature, you may pass in the `--supervise` flag to `spark-submit` when launching your
    +application. Then, if you wish to kill an application that is failing repeatedly, you may do so
    --- End diff --
    
    but this whole paragraph is about cluster mode


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-65874938
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24197/
    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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21561337
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application on failure. To use
    --- End diff --
    
    Additionally, the `cluster` mode can be optionally asked to automatically restart your application on failure (that is, if process exits with non-zero exit code).


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66520544
  
    JK I just back ported this into branch-1.2 and 1.1.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21582109
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application automatically if it
    --- End diff --
    
    nit: my point was also that it is known that we are talking about "standalone"  so you can just say "cluster mode". But its okay. 


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21561116
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application on failure. To use
    --- End diff --
    
    Could you define what is meant by failure? How is failure detected. Non-zero exit code?


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66399537
  
      [Test build #24284 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24284/consoleFull) for   PR 3627 at commit [`9ca0908`](https://github.com/apache/spark/commit/9ca0908e632fc7434aea1a98511bd5edd1c744ff).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66393716
  
      [Test build #24284 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24284/consoleFull) for   PR 3627 at commit [`9ca0908`](https://github.com/apache/spark/commit/9ca0908e632fc7434aea1a98511bd5edd1c744ff).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21562628
  
    --- Diff: docs/spark-standalone.md ---
    @@ -272,6 +272,15 @@ should specify them through the `--jars` flag using comma as a delimiter (e.g. `
     To control the application's configuration or execution environment, see
     [Spark Configuration](configuration.html).
     
    +Additionally, standalone `cluster` mode supports restarting your application on failure. To use
    +this feature, you may pass in the `--supervise` flag to `spark-submit` when launching your
    +application. Then, if you wish to kill an application that is failing repeatedly, you may do so
    --- End diff --
    
    ...kill an application running in the `cluster` mode...


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66399541
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24284/
    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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-65874932
  
      [Test build #24197 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24197/consoleFull) for   PR 3627 at commit [`2b55ed2`](https://github.com/apache/spark/commit/2b55ed22848b32c3bbdf39ec0091f621351cdb22).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/spark/pull/3627


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3627#discussion_r21406531
  
    --- Diff: docs/spark-standalone.md ---
    @@ -257,7 +257,7 @@ To run an interactive Spark shell against the cluster, run the following command
     
     You can also pass an option `--total-executor-cores <numCores>` to control the number of cores that spark-shell uses on the cluster.
     
    -# Launching Compiled Spark Applications
    +# Launching Spark Applications
    --- End diff --
    
    I checked that there are no references to this section


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66398746
  
    LGTM.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-66520113
  
    Ok merging into master 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-4771][Docs] Document standalone cluster...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/3627#issuecomment-65867988
  
      [Test build #24197 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24197/consoleFull) for   PR 3627 at commit [`2b55ed2`](https://github.com/apache/spark/commit/2b55ed22848b32c3bbdf39ec0091f621351cdb22).
     * This patch merges cleanly.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org