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

[email protected] commented on HADOOP-7314:
-------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/775/#review775
-----------------------------------------------------------

Ship it!


+1 Looks good.

- Tom


On 2011-05-23 15:49:39, Jeffrey Naisbitt wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/775/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-05-23 15:49:39)
bq.  
bq.  
bq.  Review request for hadoop-common and hadoop-mapreduce.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  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.
bq.  
bq.  
bq.  This addresses bug HADOOP-7314.
bq.      https://issues.apache.org/jira/browse/HADOOP-7314
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/src/java/org/apache/hadoop/net/DNSToSwitchMapping.java 1125067 
bq.    trunk/src/java/org/apache/hadoop/net/ScriptBasedMapping.java 1125067 
bq.    trunk/src/test/core/org/apache/hadoop/net/StaticMapping.java 1125067 
bq.    trunk/src/test/core/org/apache/hadoop/net/TestScriptBasedMapping.java 
1125067 
bq.    trunk/src/java/org/apache/hadoop/net/CachedDNSToSwitchMapping.java 
1125067 
bq.  
bq.  Diff: https://reviews.apache.org/r/775/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Jeffrey
bq.  
bq.



> 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
>            Reporter: Jeffrey Naisbitt
>            Assignee: Jeffrey Naisbitt
>         Attachments: 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