This is an automated email from the ASF dual-hosted git repository.

snuyanzin pushed a commit to branch release-2.3
in repository https://gitbox.apache.org/repos/asf/flink.git

commit 3cf3c9315980f14825fdff8a3feae14d99d9a234
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.
    
    Generated-by: Claude Fable 5
    (cherry picked from commit 1a2f45861c19ff06bd6e5e2565c823ec128cf266)
---
 .../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 144aeb9d1c6..a633c59dd05 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
@@ -173,7 +173,7 @@ public class SavepointDeepCopyTest extends 
AbstractTestBaseJUnit4 {
                 .write(savepointPath2);
         env.execute("create savepoint2");
 
-        Set<String> stateFiles2 = 
getFileNamesInDirectory(Paths.get(savepointPath1));
+        Set<String> stateFiles2 = 
getFileNamesInDirectory(Paths.get(savepointPath2));
 
         Assert.assertTrue(
                 "Failed to create savepoint2 from savepoint1 with additional 
state files",

Reply via email to