[ 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