You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by aljoscha <gi...@git.apache.org> on 2017/10/02 10:33:11 UTC

[GitHub] flink issue #4747: [FLINK-7728] [DataStream] Flush StatusWatermarkValve once...

Github user aljoscha commented on the issue:

    https://github.com/apache/flink/pull/4747
  
    This looks very good, the test is also much cleaner and understandable now. 👌 
    
    One thing, in the refactoring the bit of testing code that verifies the fix for FLINK-7721 was dropped (I checked this by commenting out the fix in the code). I'll re-add a test for that and merge.


---