GitHub user tillrohrmann opened a pull request:

    https://github.com/apache/flink/pull/3446

    [FLINK-5940] [checkpoint] Harden ZooKeeperCompletedCheckpointStore.recover 
method

    Before ZooKeeperCompletedCheckpointStore only tried to recover the latest 
completed
    checkpoint even though it might have read older checkpoint state handles 
from
    ZooKeeper. In order to deal with corrupted state handles, this commit 
changes the
    behaviour such that the completed checkpoint store adds all read retrievable
    state handles from ZooKeeper and upon request of the latest checkpoint it 
will
    return the latest completed checkpoint which could be retrieved from the 
state
    handles. Broken state handles are removed from the completed checkpoint 
store and
    ZooKeeper.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tillrohrmann/flink fixCheckpointRecovery

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3446.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3446
    
----
commit e8fe16004562c012f15f2f1efc0ad45157fbb9dd
Author: Till Rohrmann <trohrm...@apache.org>
Date:   2017-03-01T13:08:35Z

    [FLINK-5940] [checkpoint] Harden ZooKeeperCompletedCheckpointStore.recover 
method
    
    The ZooKeeperCompletedCheckpointStore only tries to recover the latest 
completed
    checkpoint even though it might have read older checkpoint state handles 
from
    ZooKeeper. In order to deal with corrupted state handles, this commit 
changes the
    behaviour such that the completed checkpoint store adds all read retrievable
    state handles from ZooKeeper and upon request of the latest checkpoint it 
will
    return the latest completed checkpoint which could be retrieved from the 
state
    handles. Broken state handles are removed from the completed checkpoint 
store and
    ZooKeeper.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to