[ 
https://issues.apache.org/jira/browse/YARN-628?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Siddharth Seth updated YARN-628:
--------------------------------

    Attachment: YARN-628.txt

The patch makes the following changes
- Changes RPCUtil.unwrapAndThrowException to throw IOException and 
YarnRemoteException.
- Fixes the implementation of this Unwrap method.
- Removed references to UndeclaredThrowableException from YARN tests.
- Adds unit tests for RPCUtil.unwrapAndThrowException

unwrapAndThrowException was earlier returning an exception. This has been 
changed to throw an exception instead - seems like a cleaner approach since 
individual PBClientImpls don't need an additional check on the exception type 
before throwing. It does introduce this strange bit of code in all PBImpls 
'return null', which will never be executed.
                
> Fix YarnException unwrapping
> ----------------------------
>
>                 Key: YARN-628
>                 URL: https://issues.apache.org/jira/browse/YARN-628
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 2.0.4-alpha
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: YARN-628.txt
>
>
> Unwrapping of YarnRemoteExceptions (currently in YarnRemoteExceptionPBImpl, 
> RPCUtil post YARN-625) is broken, and often ends up throwin 
> UndeclaredThrowableException. This needs to be fixed.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to