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

Jean-Marc Spaggiari updated HBASE-7467:
---------------------------------------

    Attachment: HBASE-7465_7467-v3.patch

Attached is Jesse's patch + mine both in a single file. But they can still noth 
be applied separatly. I'm not sure what's the best way to go, so I will let you 
pickup the recommanded one.

Also, I'm wondering if I should replace
if (children == null) return fs.delete(toCheck, false);

by 
if (children == null) {
  try {
    return fs.delete(toCheck, false);
  } catch (IOException e) {
    if (LOG.isTraceEnabled()) {
      LOG.trace("Couldn't delete directory: " + toCheck, e);
    }
  }
  // couldn't delete w/o exception, so we can't return success.
  return false;
}

Like Jesse did too...
                
> CleanerChore checkAndDeleteDirectory not deleting empty directories.
> --------------------------------------------------------------------
>
>                 Key: HBASE-7467
>                 URL: https://issues.apache.org/jira/browse/HBASE-7467
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Jean-Marc Spaggiari
>            Priority: Minor
>             Fix For: 0.96.0, 0.94.5
>
>         Attachments: HBASE-7465_7467-v3.patch, HBASE-7467.patch, 
> HBASE-7467-v2.patch
>
>
> CleanerChore checkAndDeleteDirectory is not deleting empty directories. As a 
> result, some directories are kept in the FS but should have been removed.
> To reproduce, simply create an empty directory under 
> /hbase/.archive/table_name/. If you place a file into this directory, it's 
> not more empty and therefore it's correctly removed.

--
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

Reply via email to