[jira] [Resolved] (HBASE-7465) CleanerChore shouldn't leak IOException for failed delete of non-empty directory

["Jesse Yates (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-7465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jesse Yates resolved HBASE-7465.
--------------------------------

    Resolution: Duplicate

This work got picked up in HBASE-7467, so just duping it and moving on.
                
> CleanerChore shouldn't leak IOException for failed delete of non-empty directory
> --------------------------------------------------------------------------------
>
>                 Key: HBASE-7465
>                 URL: https://issues.apache.org/jira/browse/HBASE-7465
>             Project: HBase
>          Issue Type: Bug
>          Components: master
>    Affects Versions: 0.96.0, 0.94.4
>            Reporter: Jesse Yates
>            Assignee: Jesse Yates
>             Fix For: 0.96.0, 0.94.5
>
>         Attachments: hbase-7465-v0.patch
>
>
> CleanerChore has the following lines:
> {code}
>     // if all the children have been deleted, then we should try to delete this directory. However,
>     // don't do so recursively so we don't delete files that have been added since we checked.
>     return canDeleteThis ? fs.delete(toCheck, false) : false;
> {code}
> has a race condition where the directory can have a file added underneath of it between being determined 'ok to delete' and not. As the comment mentions, we don't do a recursive delete to avoid deleting those files. However, the non-recursive delete of the directory throws an IOException.
> This is from http://search-hadoop.com/m/MUMTb13obDf2/CleanerChore+exception/v=plain

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira