Github user sihuazhou commented on a diff in the pull request:
https://github.com/apache/flink/pull/5966#discussion_r186769177
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyServer.java
---
@@ -170,8 +171,8 @@ public void initChannel(SocketChannel channel) throws
Exception {
localAddress = (InetSocketAddress)
bindFuture.channel().localAddress();
- long end = System.currentTimeMillis();
- LOG.info("Successful initialization (took {} ms). Listening on
SocketAddress {}.", (end - start),
bindFuture.channel().localAddress().toString());
+ final long duration = (start - System.nanoTime()) / 1_000_000;
+ LOG.info("Successful initialization (took {} ms). Listening on
SocketAddress {}.", duration, localAddress);
--- End diff --
Is this should be `final long duration = (System.nanoTime() - start) /
1_000_000;`
---
