[ 
https://issues.apache.org/jira/browse/MAPREDUCE-4067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13655731#comment-13655731
 ] 

Vinod Kumar Vavilapalli commented on MAPREDUCE-4067:
----------------------------------------------------

The patch looks good to me, except for a System.out.println in TestMRJobClient, 
which isn't bad.

+1. Will do a quick fresh-compilation and check this in.
                
> Replace YarnRemoteException with IOException in MRv2 APIs
> ---------------------------------------------------------
>
>                 Key: MAPREDUCE-4067
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4067
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Jitendra Nath Pandey
>            Assignee: Xuan Gong
>            Priority: Critical
>         Attachments: MAPREDUCE-4067.1.patch, MAPREDUCE-4067.2.patch, 
> MAPREDUCE-4067.3.patch, MAPREDUCE-4067.4.patch, MR4067.txt
>
>
> YarnRemoteException is defined as a generic wrapper for all the exceptions in 
> yarn. I think we should instead throw IOExceptions in the API, which can 
> later be extended for more specialized exceptions without breaking 
> compatibility.

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