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

    https://github.com/apache/flink/pull/5224#discussion_r159644962
  
    --- Diff: 
flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendPackageProgramTest.java
 ---
    @@ -60,7 +62,10 @@ public static void init() {
        @Test
        public void testNonExistingJarFile() {
                try {
    -                   CliFrontend frontend = new 
CliFrontend(CliFrontendTestUtils.getConfigDir());
    +                   CliFrontend frontend = new CliFrontend(
    --- End diff --
    
    Just an observation: The diff is only big because the instance under test 
is not an instance variable, and set up in a `@Before` annotated method. 
      


---

Reply via email to