[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_r159243077
  
    --- Diff: 
flink-yarn-tests/src/test/java/org/apache/flink/yarn/YARNSessionFIFOITCase.java 
---
    @@ -272,6 +275,7 @@ public void testJavaAPI() throws Exception {
                LOG.info("Shutting down cluster. All tests passed");
                // shutdown cluster
                yarnCluster.shutdown();
    +           clusterDescriptor.close();
    --- End diff --
    
    Closing in finally or try-with-resource not needed?


---