You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by StefanRRichter <gi...@git.apache.org> on 2017/10/23 03:31:34 UTC

[GitHub] flink issue #4879: [FLINK-7783] Don't always remove checkpoints in ZooKeeper...

Github user StefanRRichter commented on the issue:

    https://github.com/apache/flink/pull/4879
  
    I think overall this fix looks very good now. This is implementing a different strategy to deal with problematic checkpoints that can recover from more scenarios than the initial PR which was discussed between @aljoscha and me offline, so I prefer this approach.
    
    I only had minor comments and one talking point about potential concurrent modifications. If this turns out to be an non-issue (as I expect), I would approve this for merging.


---