[ 
https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15613251#comment-15613251
 ] 

Chris Trezzo edited comment on YARN-5027 at 10/27/16 9:28 PM:
--------------------------------------------------------------

Does the current patch have the potential to leak 
{{application\_\*\_\*\_DEL\_\*}} directories? In 
{{ResourceLocalizationService#cleanupLogDir}} would we want to check if the 
renamed \_DEL\_ directory exists in the case where the original log directory 
doesn't exist and delete if necessary?


was (Author: ctrezzo):
Does the current patch have the potential to leak 
{{application\_\*\_\*\_DEL\_\*}} directories? In 
{{ResourceLocalizationService#cleanupLogDir}} would we want to check if the 
renamed _DEL_ directory exists in the case where the original log directory 
doesn't exist and delete if necessary?

> NM should clean up app log dirs after NM restart 
> -------------------------------------------------
>
>                 Key: YARN-5027
>                 URL: https://issues.apache.org/jira/browse/YARN-5027
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>            Reporter: sandflee
>            Assignee: sandflee
>              Labels: oct16-easy
>         Attachments: YARN-5027.01.patch
>
>
> If nm start without recovery enabled, there may be many deprecated app log 
> dir in NM log dirs, NM should clean up them.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to