[
https://issues.apache.org/jira/browse/YARN-9362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16982851#comment-16982851
]
Szilard Nemeth commented on YARN-9362:
--------------------------------------
Latest patch looks good to me, +1, committed to trunk.
Thanks [~denes.gerencser] for your contribution.
Thanks [~pbacsko] for the review.
> 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
> 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]
