xintongsong commented on a change in pull request #16636:
URL: https://github.com/apache/flink/pull/16636#discussion_r680876862



##########
File path: 
flink-tests/src/test/java/org/apache/flink/test/state/operator/restore/AbstractOperatorRestoreTestBase.java
##########
@@ -177,6 +177,9 @@ private String migrateJob(ClusterClient<?> clusterClient, 
Deadline deadline) thr
                 JobStatus.CANCELED,
                 jobCanceledFuture.get(deadline.timeLeft().toMillis(), 
TimeUnit.MILLISECONDS));
 
+        // TODO: the savepoint _metadata file for the current branch is stored 
in this
+        //  savepointPath, please create the corresponding test resource 
directory and
+        //  copy the _metadata file by hand.

Review comment:
       I'd suggest to make this change a separate hotfix commit. Additionally, 
we probably should move this to the JavaDoc of the class.




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