You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Congxian Qiu (Jira)" <ji...@apache.org> on 2022/10/27 02:32:00 UTC

[jira] [Resolved] (FLINK-29157) Clarify the contract between CompletedCheckpointStore and SharedStateRegistry

     [ https://issues.apache.org/jira/browse/FLINK-29157?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Congxian Qiu resolved FLINK-29157.
----------------------------------
    Resolution: Fixed

merged into master 63767c5ed91642c67f97d9f16ff2b8955f9ae421

> Clarify the contract between CompletedCheckpointStore and SharedStateRegistry
> -----------------------------------------------------------------------------
>
>                 Key: FLINK-29157
>                 URL: https://issues.apache.org/jira/browse/FLINK-29157
>             Project: Flink
>          Issue Type: Technical Debt
>          Components: Runtime / Checkpointing
>    Affects Versions: 1.16.0, 1.15.2
>            Reporter: Roman Khachatryan
>            Assignee: Yanfei Lei
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.17.0, 1.15.3, 1.16.1
>
>
> After FLINK-24611, CompletedCheckpointStore is required to call SharedStateRegistry.unregisterUnusedState() on checkpoint subsumption and shutdown.
> Although it's not clear whether CompletedCheckpointStore is internal there are in fact external implementations (which weren't updated accordingly).
>  
> After FLINK-25872, CompletedCheckpointStore also must call checkpointsCleaner.cleanSubsumedCheckpoints.
>  
> Another issue with a custom implementation was using different java objects for state for CheckpointStore and SharedStateRegistry (after FLINK-24086). 
>  
> So it makes sense to:
>  * clarify the contract (different in 1.15 and 1.16)
>  * require using the same checkpoint objects by SharedStateRegistryFactory and CompletedCheckpointStore
>  * mark the interface(s) as PublicEvolving



--
This message was sent by Atlassian Jira
(v8.20.10#820010)