[ https://issues.apache.org/jira/browse/FLINK-5113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15735506#comment-15735506 ]
ASF GitHub Bot commented on FLINK-5113: --------------------------------------- Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/2939#discussion_r91721132 --- Diff: flink-tests/src/test/java/org/apache/flink/test/checkpointing/CoStreamCheckpointingITCase.java --- @@ -323,26 +329,25 @@ public PrefixCount map(String value) { } @Override - public Long snapshotState(long checkpointId, long checkpointTimestamp) { - return count; + public void close() throws IOException { + counts[getRuntimeContext().getIndexOfThisSubtask()] = count; --- End diff -- Please retain the order of the original methods; snapshot -> restore -> close > Make all Testing Functions implement CheckpointedFunction Interface. > -------------------------------------------------------------------- > > Key: FLINK-5113 > URL: https://issues.apache.org/jira/browse/FLINK-5113 > Project: Flink > Issue Type: Improvement > Components: DataStream API > Affects Versions: 1.2.0 > Reporter: Kostas Kloudas > Assignee: Kostas Kloudas > Fix For: 1.2.0 > > > Currently stateful functions implement the (old) Checkpointed interface. > This is issue aims at porting all these function to the new > CheckpointedFunction interface, so that they can leverage the new > capabilities by it. -- This message was sent by Atlassian JIRA (v6.3.4#6332)