dmvk commented on pull request #16535: URL: https://github.com/apache/flink/pull/16535#issuecomment-888249439
> A unit test that tests the shutdown behaviour of the SchedulerBase (e.g. make sure that the checkpoint services are shut down on closeAsync) Stressed by `DefaultSchedulerCheckpointCoordinatorTest` > A unit test making sure that the SchedulerBase is shut down if the JobMaster stops if such a test is not existing This is stressed by `org.apache.flink.runtime.jobmaster.JobMasterTest#testJobMasterOnlyTerminatesAfterTheSchedulerHasClosed` > A unit test guarding the new cleanUpJobGraph and cleanUpRemainingJobData contract. One could instantiate a TestingJobMasterService that signals that its job is finished and then we should make sure that we first remove the job from the JobGraphStore, then close the JobMasterService and then clean up the remaining HA bookkeeping data. Stressed by: ``` org.apache.flink.runtime.dispatcher.DispatcherTest#testJobDataAreCleanedUpInCorrectOrderOnFinishedJob org.apache.flink.runtime.dispatcher.DispatcherTest#testJobDataAreCleanedUpInCorrectOrderOnFailedJob org.apache.flink.runtime.dispatcher.DispatcherTest#testFailingJobManagerRunnerCleanup ``` -- 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]
