diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 522ec47f704..99320cbfb28 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -602,9 +602,6 @@ Release 2.7.0 - UNRELEASED HADOOP-9869. Configuration.getSocketAddr()/getEnum() should use getTrimmed(). (Tsuyoshi Ozawa via aajisaka) - HADOOP-11589. NetUtils.createSocketAddr should trim the input URI. - (Rakesh R via ozawa) - OPTIMIZATIONS HADOOP-11323. WritableComparator#compare keeps reference to byte array. diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetUtils.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetUtils.java index e475149aaa4..ef1092bb2f7 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetUtils.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/NetUtils.java @@ -189,7 +189,6 @@ public class NetUtils { throw new IllegalArgumentException("Target address cannot be null." + helpText); } - target = target.trim(); boolean hasScheme = target.contains("://"); URI uri = null; try { diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java index b7c76ebe1ee..a3675537bf0 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java @@ -1480,16 +1480,6 @@ public class TestConfiguration extends TestCase { // it's expected behaviour. } - public void testTrimCreateSocketAddress() { - Configuration conf = new Configuration(); - NetUtils.addStaticResolution("host", "127.0.0.1"); - final String defaultAddr = "host:1 "; - - InetSocketAddress addr = NetUtils.createSocketAddr(defaultAddr); - conf.setSocketAddr("myAddress", addr); - assertEquals(defaultAddr.trim(), NetUtils.getHostPortString(addr)); - } - public static void main(String[] argv) throws Exception { junit.textui.TestRunner.main(new String[]{ TestConfiguration.class.getName()