Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/6376#discussion_r203978068
--- Diff:
flink-tests/src/test/java/org/apache/flink/test/checkpointing/EventTimeWindowCheckpointingITCase.java
---
@@ -299,12 +288,17 @@ public void apply(
sum +=
value.f1.value;
key = value.f0;
}
- out.collect(new
Tuple4<>(key, window.getStart(), window.getEnd(), new IntType(sum)));
+
+ final Tuple4<Long,
Long, Long, IntType> result =
+ new
Tuple4<>(key, window.getStart(), window.getEnd(), new IntType(sum));
+ out.collect(result);
}
})
- .addSink(new ValidatingSink(numKeys,
numElementsPerKey / windowSize)).setParallelism(1);
+ .addSink(new ValidatingSink<>(
+ new
SinkValidatorUpdateFun(numElementsPerKey),
+ new SinkValidatorCheckFun(numKeys,
numElementsPerKey, windowSize))).setParallelism(1);
- tryExecute(env, "Tumbling Window Test");
+ env.execute("Tumbling Window Test");
--- End diff --
Because it does not require to use `SuccessExceptions` because in event
time the end of the source function is deterministic.
---
