Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/4358#discussion_r137302842
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/blob/BlobServer.java ---
@@ -174,7 +176,6 @@ public ServerSocket createSocket(int port) throws
IOException {
// start the server thread
setName("BLOB Server listener at " + getPort());
setDaemon(true);
- start();
--- End diff --
Why did you pull `start` out of the constructor? Wouldn't one always want
to start the `BlobServer` when creating it?
---
