ljz2051 commented on code in PR #24873:
URL: https://github.com/apache/flink/pull/24873#discussion_r1622189692


##########
flink-runtime/src/main/java/org/apache/flink/runtime/state/CheckpointStreamFactory.java:
##########
@@ -89,6 +91,13 @@ default void reusePreviousStateHandle(Collection<? extends 
StreamStateHandle> pr
      * @return true if it can be reused.
      */
     default boolean couldReuseStateHandle(StreamStateHandle stateHandle) {
+
+        // By default, the CheckpointStreamFactory doesn't support 
snapshot-file-merging, so the
+        // SegmentFileStateHandle type of stateHandle can not be reused.
+        if (stateHandle instanceof SegmentFileStateHandle) {
+            return false;
+        }

Review Comment:
   Yes, you are right. I have resolved it.



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to