GitHub user vanzin opened a pull request:

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

    [SPARK-17742][core] Fail launcher app handle if child process exits with 
error.

    This is a follow up to cba826d0; that commit set the app handle state
    to "LOST" when the child process exited, but that can be ambiguous. This
    change sets the state to "FAILED" if the exit code was non-zero and
    the handle state wasn't a failure state, or "LOST" if the exit status
    was zero.


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

    $ git pull https://github.com/vanzin/spark SPARK-17742

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

    https://github.com/apache/spark/pull/19012.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 #19012
    
----
commit d6514874beba87d48c01fc411775f4ecaed03a69
Author: Marcelo Vanzin <[email protected]>
Date:   2017-08-21T21:19:26Z

    [SPARK-17742][core] Fail launcher app handle if child process exits with 
error.
    
    This is a follow up to cba826d0; that commit set the app handle state
    to "LOST" when the child process exited, but that can be ambiguous. This
    change sets the state to "FAILED" if the exit code was non-zero and
    the handle state wasn't a failure state, or "LOST" if the exit status
    was zero.

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to