[
https://issues.apache.org/jira/browse/YARN-9362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16982873#comment-16982873
]
Hudson commented on YARN-9362:
------------------------------
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #17699 (See
[https://builds.apache.org/job/Hadoop-trunk-Commit/17699/])
YARN-9362. Code cleanup in TestNMLeveldbStateStoreService. Contributed
(snemeth: rev 828ab400eea64ebb628a36cc3d0d53de0bf38934)
* (edit)
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/recovery/TestNMLeveldbStateStoreService.java
> Code cleanup in TestNMLeveldbStateStoreService
> ----------------------------------------------
>
> Key: YARN-9362
> URL: https://issues.apache.org/jira/browse/YARN-9362
> Project: Hadoop YARN
> Issue Type: Improvement
> Reporter: Szilard Nemeth
> Assignee: Denes Gerencser
> Priority: Minor
> Fix For: 3.3.0
>
> Attachments: YARN-9362.001.patch, YARN-9362.002.patch
>
>
> There are many ways to improve TestNMLeveldbStateStoreService:
> 1. RecoveredContainerState fields are asserted many times repeatedly. Some
> simple method extractions would definitely make this more readable.
> 2. The tests are very long and hard to read in general: Again, finding how
> methods could be extracted to avoid code repetition could help.
> 3. You name it.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
