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

    https://github.com/apache/flink/pull/5925#discussion_r184619113
  
    --- Diff: 
flink-end-to-end-tests/flink-datastream-allround-test/src/main/java/org/apache/flink/streaming/tests/SemanticsCheckMapper.java
 ---
    @@ -21,17 +21,15 @@
     import org.apache.flink.api.common.functions.RichFlatMapFunction;
     import org.apache.flink.api.common.state.ValueState;
     import org.apache.flink.api.common.state.ValueStateDescriptor;
    -import org.apache.flink.runtime.state.FunctionInitializationContext;
    -import org.apache.flink.runtime.state.FunctionSnapshotContext;
    -import org.apache.flink.streaming.api.checkpoint.CheckpointedFunction;
    +import org.apache.flink.configuration.Configuration;
     import org.apache.flink.util.Collector;
     
     import java.io.Serializable;
     
     /**
      * This mapper validates exactly-once and at-least-once semantics in 
connection with {@link SequenceGeneratorSource}.
      */
    -public class SemanticsCheckMapper extends RichFlatMapFunction<Event, 
String> implements CheckpointedFunction {
    +public class SemanticsCheckMapper extends RichFlatMapFunction<Event, 
String> {
    --- End diff --
    
    Actually the changes here in this file are not strictly required for 
exactly-once.
    I only changed this as part of trying some things out.
    Can revert this if you prefer, @StefanRRichter.


---

Reply via email to