ASF GitHub Bot commented on YARN-8057:

GitHub user lzh3636 opened a pull request:


    YARN-8057 Inadequate information for handling catch clauses

    The description of the problem:
    I added stack traces information to those two logging statements, so that 
the full exception information can be generated to the logs.

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

    $ git pull https://github.com/lzh3636/hadoop YARN-8057

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


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

    This closes #362
commit 002935625784226ba915e8631a0c8298d423677f
Author: lzh3636 <lizi422@...>
Date:   2018-04-10T15:03:54Z

    update stack traces to show exception info


> Inadequate information for handling catch clauses
> -------------------------------------------------
>                 Key: YARN-8057
>                 URL: https://issues.apache.org/jira/browse/YARN-8057
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: api, yarn
>    Affects Versions: 3.0.0
>            Reporter: Zhenhao Li
>            Priority: Major
>              Labels: easyfix
> Their are some situations that different exception types are caught, but the 
> handling of those exceptions can not show the differences of those types. 
> Here are the code snippets we found which have this problem:
> *org/apache/hadoop/yarn/client/api/impl/NMClientImpl.java*
> [https://github.com/apache/hadoop/blob/c02d2ba50db8a355ea03081c3984b2ea0c375a3f/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/NMClientImpl.java]
> At Line *125* and Line *129.* We can see that two exception types are caught, 
> but the logging statements here can not show the exception type at all. It 
> may cause confusions to the person who is reading the log, the person can not 
> know what exception happened here.
> Maybe adding stack trace information to these two logging statements is a 
> simple way to improve it.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to