GitHub user codeboyyong opened a pull request:

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

    [SPARK-1516]Throw exception in yarn client instead of run system.exit 
directly.

    All the changes is in  the package of "org.apache.spark.deploy.yarn":
        1) Throw exception in ClinetArguments and ClientBase instead of exit 
directly.
        2) in Client's main method, if exception is caught, it will exit with 
code 1, otherwise exit with code 0.
    
    After the fix, if user integrate the spark yarn client into their 
applications, when the argument is wrong or the running is finished, the 
application won't be terminated. 


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

    $ git pull https://github.com/AlpineNow/spark 
jira_1516_systemexit_inyarnclient

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

    https://github.com/apache/spark/pull/490.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 #490
    
----
commit f7a037988a9477b2a4e7b8fce0720e5ae3eda0ac
Author: John Zhao <[email protected]>
Date:   2014-04-22T22:58:35Z

    [SPARK-1516]Throw exception in yarn clinet instead of run system.exit 
directly.
    All the changes is in  the package of "org.apache.spark.deploy.yarn":
    1) Add a ClientException with an exitCode
    2) Throws exception in ClinetArguments and ClientBase instead of exit 
directly
    3) in Client's main method, catch exception and exit with the exitCode.
    
    After the fix, if user integrate the spark yarn cline into their 
applications, when the argument is wrong or the running is finished, the 
application will not exit. Now the exit only happens in command line running.

commit 5ae631d688d25a2f1ca79319c490357a3d4caad4
Author: John Zhao <[email protected]>
Date:   2014-04-23T00:45:19Z

    Code formatting

----


---
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.
---

Reply via email to