zentol commented on issue #8243: [FLINK-12299] ExecutionConfig#setAutoWatermarkInterval should check param(interval should not less than zero) URL: https://github.com/apache/flink/pull/8243#issuecomment-495145354 It is true that it's difficult to find a valid use-case for the interval being 0, but the sad reality is that all existing classes handle this case gracefully and there might be someone out there relying on it, possibly even just in tests (like we do). Hence I'd rather not change that. Will merge the PR as is once travis is green.
---------------------------------------------------------------- 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] With regards, Apache Git Services
