AlexYinHan commented on code in PR #23514:
URL: https://github.com/apache/flink/pull/23514#discussion_r1402953830


##########
flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/filemerging/FileMergingSnapshotManagerBase.java:
##########
@@ -206,6 +213,64 @@ protected PhysicalFile createPhysicalFile(SubtaskKey 
subtaskKey, CheckpointedSta
                 "Could not open output stream for state file merging.", 
latestException);
     }
 
+    @Override
+    public FileMergingCheckpointStateOutputStream 
createCheckpointStateOutputStream(

Review Comment:
   It should be created by a ``` FsCheckpointStreamFactory```, similar to 
```FsCheckpointStateOutputStream```. 
   
   I have added a class named ```FsMergingCheckpointStorageLocation```, whose 
object is created by ```FsMergingCheckpointStorageAccess``` and creates 
```FileMergingCheckpointStateOutputStream```.



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

Reply via email to