[
https://issues.apache.org/jira/browse/YARN-4994?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15676823#comment-15676823
]
Andras Bokor edited comment on YARN-4994 at 11/29/16 2:07 PM:
--------------------------------------------------------------
[~jhung],
Thanks for taking a look on it. I rebased my patch. The [~hadoopqa] -1s seem
unrelated:
* The two JUnit fail seems unrelated and will be fixed with YARN-5728 and
YARN-5851.
* The checkstyle warning is unrelated.
* Findbugs issue seems the same problem as YARN-5138.
was (Author: boky01):
[~jhung],
Thanks for taking a look on it. I rebased my patch. The [~hadoopqa] -1s seem
unrelated:
* The two JUnit fail seems unrelated and will be fixed with YARN-5728 and
YARN-5851.
* The checkstyle warning is unrelted.
* Findbugs issue seems the same problem as YARN-5138.
> Use MiniYARNCluster with try-with-resources in tests
> ----------------------------------------------------
>
> Key: YARN-4994
> URL: https://issues.apache.org/jira/browse/YARN-4994
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: test
> Affects Versions: 2.7.0
> Reporter: Andras Bokor
> Assignee: Andras Bokor
> Priority: Trivial
> Labels: oct16-easy
> Fix For: 2.7.0
>
> Attachments: HDFS-10287.01.patch, HDFS-10287.02.patch,
> HDFS-10287.03.patch, YARN-4994.04.patch, YARN-4994.05.patch,
> YARN-4994.06.patch, YARN-4994.07.patch, YARN-4994.08.patch, YARN-4994.09.patch
>
>
> In tests MiniYARNCluster is used with the following pattern:
> In try-catch block create a MiniYARNCluster instance and in finally block
> close it.
> [Try-with-resources|https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html]
> is preferred since Java7 instead of the pattern above.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
