Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/5966#discussion_r186800971
--- 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 --
You are absolutely right, thanks for catching this!
---
