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

Ted Yu commented on HBASE-19969:
--------------------------------

{code}
+        if (!fs.delete(destPath, true)) {
+          System.out.println("Failed to delete " + destPath);
+        }
+        boolean res = fs.rename(tmpPath, destPath);
{code}
As I said previously, if the delete() call fails, the rename() would fail as 
well.
I think IOE can be thrown for both failures.

getBackupTmpDirForBackupId is no longer in the patch. Can you describe related 
changes ?







> Improve FT in merge operation
> -----------------------------
>
>                 Key: HBASE-19969
>                 URL: https://issues.apache.org/jira/browse/HBASE-19969
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>            Priority: Major
>         Attachments: HBASE-19969-v1.patch, HBASE-19969-v2.patch
>
>
> Some file system operations are not fault tolerant during merge. We delete 
> backup data in a backup file system, then copy new data over to backup 
> destination. Deletes can be partial, copy can fail as well



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to