kl0u commented on a change in pull request #13509:
URL: https://github.com/apache/flink/pull/13509#discussion_r497333033
##########
File path:
flink-core/src/main/java/org/apache/flink/api/common/ExecutionConfig.java
##########
@@ -126,7 +126,7 @@
private boolean autoTypeRegistrationEnabled = true;
private boolean forceAvro = false;
- private long autoWatermarkInterval = 0;
+ private long autoWatermarkInterval = 200;
Review comment:
I do not think it contributes to anything for now. It is just that if in
the future someone, for whatever reason, wants to do a check like: `if
(interval == DEFAULT) ...`, or he/she wants to reset the interval to the
default, the literal `200` would be a bit obscure instead of a
`DEFAULT_WM_INTERVAL`. But again, it does not add any value for now.
----------------------------------------------------------------
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]
