Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6147#discussion_r195050299
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/jobgraph/JobGraphTest.java 
---
    @@ -18,16 +18,30 @@
     
     package org.apache.flink.runtime.jobgraph;
     
    -import static org.junit.Assert.*;
    -
    -import java.util.List;
    -
     import org.apache.flink.api.common.InvalidProgramException;
    +import org.apache.flink.api.common.cache.DistributedCache;
     import org.apache.flink.core.testutils.CommonTestUtils;
     import org.apache.flink.runtime.io.network.partition.ResultPartitionType;
    +
    +import org.junit.ClassRule;
     import org.junit.Test;
    +import org.junit.rules.TemporaryFolder;
    +
    +import java.io.IOException;
    +import java.nio.file.Files;
    +import java.nio.file.Path;
    +import java.util.List;
    +import java.util.Map;
    +
    +import static org.junit.Assert.assertEquals;
    +import static org.junit.Assert.assertFalse;
    +import static org.junit.Assert.assertNotNull;
    +import static org.junit.Assert.fail;
     
     public class JobGraphTest {
    --- End diff --
    
    Could extend `TestLogger`


---

Reply via email to