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-492559102
 
 
   I've reverted the formatting changes by manually adding spaces/tabs through 
the github UI, reverted unrelated changes in 
`ExecutionConfig#getRegisteredKryoTypes()/registerPojoType()`, and modified the 
check to allow 0, since 
`org.apache.flink.streaming.api.environment.StreamExecutionEnvironment#setStreamTimeCharacteristic`
 sets it to 0. Since 0 is the default value we should also accept 0 as input.

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

Reply via email to