fredia commented on code in PR #19907:
URL: https://github.com/apache/flink/pull/19907#discussion_r895935387
##########
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskLocalStateStoreImpl.java:
##########
@@ -98,6 +104,15 @@ public class TaskLocalStateStoreImpl implements
OwnedTaskLocalStateStore {
@GuardedBy("lock")
private final SortedMap<Long, TaskStateSnapshot>
storedTaskStateByCheckpointID;
+ /** The relationship between checkpoints. (cp1,cp3) means cp3 refer to
cp1's state. */
+ private final Map<Long, Long> referredByCheckpointID;
+
+ /**
+ * (MaterializationID, created by checkpointID). Because local store only
keeps one checkpoint,
+ * at most one materialization can exist at the same time.
+ */
+ private Tuple2<Long, Long> currentMaterializationID;
+
public TaskLocalStateStoreImpl(
Review Comment:
Nice suggestion, I created ChangelogTaskLocalStateStore to encapsulate
changelog-specific logic.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]