fredia commented on code in PR #25116:
URL: https://github.com/apache/flink/pull/25116#discussion_r1689498525
##########
flink-tests/src/test/java/org/apache/flink/test/checkpointing/SnapshotFileMergingCompatibilityITCase.java:
##########
@@ -344,13 +343,14 @@ private static void verifyCheckpointExistOrWaitDeleted(
}
/**
- * Traverse the checkpoint metadata and verify all the state handle is
disposed.
+ * Traverse the checkpoint metadata and verify all the state handle exist
or be disposed.
*
* @param metadata the metadata to traverse.
+ * @param exist all corresponding files of the state handle should exist
* @return true if all corresponding files are deleted.
*/
- private static boolean verifyCheckpointDisposed(CheckpointMetadata
metadata) {
- AtomicBoolean disposed = new AtomicBoolean(true);
+ private boolean verifyCheckpointExist(CheckpointMetadata metadata, boolean
exist) {
Review Comment:
Why remove the `static` mark?
--
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
