[FLINK-13692] Make CompletedCheckpointStore backwards compatible?

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Abandoned
• Affects Version/s: 1.9.0, 1.10.0
• Component/s: Runtime / Checkpointing
• Labels: auto-deprioritized-major
• Assignee: Unassigned
• Reporter: Till Rohrmann
• Created: 12/Aug/19 09:36
• Updated: 29/Sep/21 02:01
• Resolved: 29/Sep/21 02:01

Description

With FLINK-11159 we changed the signature of CompletedCheckpointStore#getLatestCheckpoint from getLatestCheckpoint() to getLatestCheckpoint(boolean). This is an API breaking change which causes users of this interface having to adapt their implementation and recompile it. Since CompletedCheckpointStore is no public API, this change is technically ok, but not so nice.

If we decide to provide backwards compatibility, then we should still keep getLatestCheckpoint() and add a deprecation annotation to it. At least, we should update the JavaDocs of the CompletedCheckpointStore#getLatestCheckpoint method to reflect what the passed parameter states.

Issue Links

• is caused by: FLINK-11159 Allow configuration whether to fall back to savepoints for restore
• relates to: FLINK-13696 Revisit & update Flink's public API annotations