tillrohrmann commented on a change in pull request #18194:
URL: https://github.com/apache/flink/pull/18194#discussion_r786532968



##########
File path: 
flink-runtime/src/test/java/org/apache/flink/runtime/dispatcher/DispatcherTest.java
##########
@@ -1045,6 +1048,27 @@ public void 
testJobDataAreCleanedUpInCorrectOrderOnFailedJob() throws Exception
         testJobDataAreCleanedUpInCorrectOrder(JobStatus.FAILED);
     }
 
+    @Test
+    public void testOnlyRecoveredJobsAreRetainedInTheBlobServer() throws 
Exception {
+        final JobID jobId1 = new JobID();
+        final JobID jobId2 = new JobID();
+        final byte[] fileContent = {1, 2, 3, 4};
+        final BlobServer blobServer = getBlobServer();
+        final PermanentBlobKey blobKey1 = blobServer.putPermanent(jobId1, 
fileContent);
+        final PermanentBlobKey blobKey2 = blobServer.putPermanent(jobId2, 
fileContent);
+
+        final TestingDispatcher dispatcher =
+                new TestingDispatcherBuilder()
+                        .setInitialJobGraphs(Collections.singleton(new 
JobGraph(jobId1, "foobar")))
+                        .build();
+
+        org.assertj.core.api.Assertions.assertThat(blobServer.getFile(jobId1, 
blobKey1))

Review comment:
       True. I will use a direct executor to solve the problem.




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