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

Jeffrey Naisbitt commented on HADOOP-7314:
------------------------------------------

After that comment, I was asked to separate the patch from MAPREDUCE-2489 so 
that the code reviewed here was all in one place.  So, the 
HADOOP-7314-0.20s.patch file is in fact for the 0.20s branch.

Sorry about the confusion.  I tried to clear it up with these comments above 
(https://issues.apache.org/jira/browse/HADOOP-7314?focusedCommentId=13065421&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13065421
 and 
https://issues.apache.org/jira/browse/HADOOP-7314?focusedCommentId=13065422&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13065422),
 but I'm guessing it got lost in the jumble of messages and updates.

Thanks for looking at this!

> Add support for throwing UnknownHostException when a host doesn't resolve
> -------------------------------------------------------------------------
>
>                 Key: HADOOP-7314
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7314
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 0.20.205.0, 0.23.0
>            Reporter: Jeffrey Naisbitt
>            Assignee: Jeffrey Naisbitt
>             Fix For: 0.23.0
>
>         Attachments: HADOOP-7314-0.20s.patch, HADOOP-7314-v2.patch, 
> HADOOP-7314-v3.patch, HADOOP-7314.patch
>
>
> As part of MAPREDUCE-2489, we need support for having the resolve methods 
> (for DNS mapping) throw UnknownHostExceptions.  (Currently, they hide the 
> exception).  Since the existing 'resolve' method is ultimately used by 
> several other locations/components, I propose we add a new 
> 'resolveValidHosts' method.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to