[ https://issues.apache.org/jira/browse/FLINK-2314?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298185#comment-15298185 ]
ASF GitHub Bot commented on FLINK-2314: --------------------------------------- Github user aljoscha commented on a diff in the pull request: https://github.com/apache/flink/pull/2020#discussion_r64392589 --- Diff: flink-tests/src/test/java/org/apache/flink/test/checkpointing/StreamFaultToleranceTestBase.java --- @@ -104,8 +106,25 @@ public void runCheckpointedProgram() { postSubmit(); } catch (Exception e) { + Throwable th = e; + int depth = 0; + + for (; depth < 20; depth++) { + if (th instanceof SuccessException) { + try { + postSubmit(); + } catch (Exception e1) { + e1.printStackTrace(); --- End diff -- Should we not forward the exception here? You introduced this block so that `postSubmit()` also runs when the `SuccessException` was thrown, right? > Make Streaming File Sources Persistent > -------------------------------------- > > Key: FLINK-2314 > URL: https://issues.apache.org/jira/browse/FLINK-2314 > Project: Flink > Issue Type: Improvement > Components: Streaming > Affects Versions: 0.9 > Reporter: Stephan Ewen > Assignee: Kostas Kloudas > > Streaming File sources should participate in the checkpointing. They should > track the bytes they read from the file and checkpoint it. > One can look at the sequence generating source function for an example of a > checkpointed source. -- This message was sent by Atlassian JIRA (v6.3.4#6332)