GitHub user tillrohrmann opened a pull request:

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

    [FLINK-5063] Discard state handles of declined or expired state handles

    Whenever the checkpoint coordinator receives an acknowledge checkpoint 
message which belongs
    to the job maintained by the checkpoint coordinator, it should either 
record the state handles
    for later processing or discard to free the resources. The latter case can 
happen if a
    checkpoint has been expired and late acknowledge checkpoint messages 
arrive. Furthermore, it
    can happen if a Task sent a decline checkpoint message while other Tasks 
where still drawing
    a checkpoint. This PR changes the behaviour such that state handles 
belonging to the job of
    the checkpoint coordinator are discarded if they could not be added to the 
PendingCheckpoint.
    
    Review @uce, @StephanEwen 

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

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

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

    https://github.com/apache/flink/pull/2812.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 #2812
    
----
commit c4c000d1b39de5617b6796eed524ce2a449100d3
Author: Till Rohrmann <trohrm...@apache.org>
Date:   2016-11-14T17:33:55Z

    [FLINK-5063] Discard state handles of declined or expired state handles
    
    Whenever the checkpoint coordinator receives an acknowledge checkpoint 
message which belongs
    to the job maintained by the checkpoint coordinator, it should either 
record the state handles
    for later processing or discard to free the resources. The latter case can 
happen if a
    checkpoint has been expired and late acknowledge checkpoint messages 
arrive. Furthremore, it
    can happen if a Task sent a decline checkpoint message while other Tasks 
where still drawing
    a checkpoint. This PR changes the behaviour such that state handles 
belonging to the job of
    the checkpoint coordinator are discarded if they could not be added to the 
PendingCheckpoint.

----


---
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