dawidwys commented on a change in pull request #17471:
URL: https://github.com/apache/flink/pull/17471#discussion_r729026251



##########
File path: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java
##########
@@ -105,9 +105,7 @@
 
     private static final Logger LOG = 
LoggerFactory.getLogger(StreamingJobGraphGenerator.class);
 
-    private static final long DEFAULT_NETWORK_BUFFER_TIMEOUT = 100L;
-
-    public static final long UNDEFINED_NETWORK_BUFFER_TIMEOUT = -1L;
+    public static final long DISABLED_NETWORK_BUFFER_TIMEOUT = -1L;

Review comment:
       I think it's a good idea to move it to one of the `Options` classes. I 
moved it to the `ExecutionOptions` where the corresponding 
`ExecutionOptions#BUFFER_TIMEOUT` is defined.




-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to