[GitHub] flink pull request #5216: [FLINK-8329] [flip6] Move YarnClient to AbstractYa...

[GJL]

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

    https://github.com/apache/flink/pull/5216#discussion_r159242946
  
    --- Diff: 
flink-yarn-tests/src/test/java/org/apache/flink/yarn/YARNITCase.java ---
    @@ -78,6 +83,9 @@ public void testPerJobMode() {
                jobGraph.addJar(new 
org.apache.flink.core.fs.Path(testingJar.toURI()));
     
                YarnClusterClient clusterClient = 
yarnClusterDescriptorV2.deployJobCluster(clusterSpecification, jobGraph);
    +
    +           clusterClient.shutdown();
    +           yarnClusterDescriptorV2.close();
    --- End diff --
    
    Closing in finally or try-with-resource not needed?


---