[GitHub] flink pull request #5092: [FLINK-4812][metrics] Expose currentLowWatermark f...

[tzulitai]

Github user tzulitai commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5092#discussion_r153737987
  
    --- Diff: 
flink-streaming-java/src/test/java/org/apache/flink/streaming/runtime/operators/StreamSourceOperatorTest.java
 ---
    @@ -224,6 +226,39 @@ public void testLatencyMarkEmission() throws Exception 
{
                Assert.assertTrue(output.get(i).isWatermark());
        }
     
    +   @Test
    +   public void testManualWatermarkContextWatermarkMetric() throws 
Exception {
    --- End diff --
    
    Probably should add verifying the metric value after emitting the 
`Long.MAX_VALUE` watermark to this test, just to be on the safe side.


---