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

Rohith Sharma K S commented on HADOOP-12687:
--------------------------------------------

Hi [~sunilg], small update can you do for the patch?
# Instead of catching UnknownHostException , can you move down {{addr = 
InetAddress.getByName(host);}} like below. So need not catch 
UnknownHostException. And add a comment there.
{code}

          addr = getByNameWithSearch(host);
          if (addr == null) {
            addr = getByExactName(host);
            if (addr == null) {
// comment
              addr = InetAddress.getByName(host);
            }
          }
        
{code}
# Not related to patch, need to change the summary of this JIRA that reflect 
actual code change in  Hadoop Common.

> Timeout for tests in TestYarnClient, TestAMRMClient and TestNMClient
> --------------------------------------------------------------------
>
>                 Key: HADOOP-12687
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12687
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Junping Du
>            Assignee: Sunil G
>              Labels: security
>         Attachments: 0001-YARN-4352.patch, 0002-YARN-4352.patch
>
>
> From 
> https://builds.apache.org/job/PreCommit-YARN-Build/9661/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_79.txt,
>  we can see the tests in TestYarnClient, TestAMRMClient and TestNMClient get 
> timeout which can be reproduced locally.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to