[ https://issues.apache.org/jira/browse/FLINK-5488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16058798#comment-16058798 ]
ASF GitHub Bot commented on FLINK-5488: --------------------------------------- Github user zjureel commented on a diff in the pull request: https://github.com/apache/flink/pull/4022#discussion_r123425557 --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterClientV2.java --- @@ -146,7 +146,9 @@ public ApplicationStatus getApplicationStatus() { @Override public void finalizeCluster() { - // Do nothing --- End diff -- I have reverted the changes and fixed another problem as Ted Yu metioned in FLINK-5488, thanks @zentol > yarnClient should be closed in AbstractYarnClusterDescriptor for error > conditions > --------------------------------------------------------------------------------- > > Key: FLINK-5488 > URL: https://issues.apache.org/jira/browse/FLINK-5488 > Project: Flink > Issue Type: Bug > Components: YARN > Reporter: Ted Yu > Assignee: Fang Yong > > Here is one example: > {code} > if(jobManagerMemoryMb > maxRes.getMemory() ) { > failSessionDuringDeployment(yarnClient, yarnApplication); > throw new YarnDeploymentException("The cluster does not have the > requested resources for the JobManager available!\n" > + "Maximum Memory: " + maxRes.getMemory() + "MB Requested: " + > jobManagerMemoryMb + "MB. " + NOTE); > } > {code} > yarnClient implements Closeable. > It should be closed in situations where exception is thrown. -- This message was sent by Atlassian JIRA (v6.4.14#64029)