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)