Junping Du commented on YARN-2167:

The patch is very tiny and straight-forward, so no need for additional unit 

> LeveldbIterator should get closed in 
> NMLeveldbStateStoreService#loadLocalizationState() within finally block
> ------------------------------------------------------------------------------------------------------------
>                 Key: YARN-2167
>                 URL: https://issues.apache.org/jira/browse/YARN-2167
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>            Reporter: Junping Du
>            Assignee: Junping Du
>         Attachments: YARN-2167.patch
> In NMLeveldbStateStoreService#loadLocalizationState(), we have 
> LeveldbIterator to read NM's localization state but it is not get closed in 
> finally block. We should close this connection to DB as a common practice. 

This message was sent by Atlassian JIRA

Reply via email to