diff --git a/CHANGES.txt b/CHANGES.txt index 577f5ee020..29b09b97bc 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -933,6 +933,9 @@ Release 0.21.0 - Unreleased (Dmytro Molkov via hairong) BUG FIXES + + HADOOP-6760. WebServer shouldn't increase port number in case of negative + port setting caused by Jetty's race (cos) HADOOP-5379. CBZip2InputStream to throw IOException on data crc error. (Rodrigo Schmidt via zshao) diff --git a/src/java/org/apache/hadoop/http/HttpServer.java b/src/java/org/apache/hadoop/http/HttpServer.java index de7c860bf0..dbd0e80a7d 100644 --- a/src/java/org/apache/hadoop/http/HttpServer.java +++ b/src/java/org/apache/hadoop/http/HttpServer.java @@ -495,32 +495,6 @@ public void start() throws IOException { } //Workaround end LOG.info("Jetty bound to port " + port); webServer.start(); - // Workaround for HADOOP-6386 - port = listener.getLocalPort(); - if (port < 0) { - LOG.warn("Bounds port is " + port + " after webserver start"); - for (int i = 0; i < MAX_RETRIES/2; i++) { - try { - webServer.stop(); - } catch (Exception e) { - LOG.warn("Can't stop web-server", e); - } - Thread.sleep(1000); - - listener.setPort(oriPort == 0 ? 0 : (oriPort += 1)); - listener.open(); - Thread.sleep(100); - webServer.start(); - LOG.info(i + "attempts to restart webserver"); - port = listener.getLocalPort(); - if (port > 0) - break; - } - if (port < 0) - throw new Exception("listener.getLocalPort() is returning " + - "less than 0 even after " +MAX_RETRIES+" resets"); - } - // End of HADOOP-6386 workaround break; } catch (IOException ex) { // if this is a bind exception,