This is an automated email from the ASF dual-hosted git repository. rkhachatryan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/flink.git
commit 1a2f45861c19ff06bd6e5e2565c823ec128cf266 Author: Aleksandr Savonin <[email protected]> AuthorDate: Fri Jul 3 19:24:56 2026 +0200 [hotfix][tests] Fix SavepointDeepCopyTest checking the source directory twice testSavepointDeepCopy listed savepointPath1 for both stateFiles1 and stateFiles2, so the copied-files assertion compared savepoint1's directory with itself and savepoint2's directory was never examined at all. Point stateFiles2 at savepointPath2, as the variable name, the assertion messages, and the test's documented steps always intended. The mix-up was introduced when the getFileNamesInDirectory helper was extracted (f125067ed0c); before that, both checks listed savepointPath2. Co-Authored-By: Claude Fable 5 <[email protected]> --- .../src/test/java/org/apache/flink/state/api/SavepointDeepCopyTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/flink-libraries/flink-state-processing-api/src/test/java/org/apache/flink/state/api/SavepointDeepCopyTest.java b/flink-libraries/flink-state-processing-api/src/test/java/org/apache/flink/state/api/SavepointDeepCopyTest.java index ab9119ad56a..9c214c92a23 100644 --- a/flink-libraries/flink-state-processing-api/src/test/java/org/apache/flink/state/api/SavepointDeepCopyTest.java +++ b/flink-libraries/flink-state-processing-api/src/test/java/org/apache/flink/state/api/SavepointDeepCopyTest.java @@ -168,7 +168,7 @@ class SavepointDeepCopyTest extends AbstractTestBase { .write(savepointPath2); env.execute("create savepoint2"); - Set<String> stateFiles2 = getFileNamesInDirectory(Paths.get(savepointPath1)); + Set<String> stateFiles2 = getFileNamesInDirectory(Paths.get(savepointPath2)); assertThat(stateFiles2) .as("Failed to create savepoint2 from savepoint1 with additional state files")
