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