cuspymd commented on a change in pull request #15704:
URL: https://github.com/apache/flink/pull/15704#discussion_r618059741



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyConfig.java
##########
@@ -49,7 +49,8 @@
 
     private final InetAddress serverAddress;
 
-    private final int serverPort;
+    private final String serverPortRange;
+    private final Iterator<Integer> serverPortRangeIterator;

Review comment:
       The point of my question is, isn't one field enough? For example, if 
there is only the `serverPortRange` field, can't the iterator be created at the 
desired time using a getter method?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to