[
https://issues.apache.org/jira/browse/HADOOP-14769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Thomas Marquardt updated HADOOP-14769:
--------------------------------------
Attachment: HADOOP-14769-002.patch
Attaching HADOOP-14769-002.patch. This is only a style change.
Thanks for feedback.
1) I will not refactor *AzureNativeFileSystemStore.java L2503-2505* at this
time. This is a targeted fix for recursive delete.
2) I removed the nesting at *NativeAzureFileSystem.java L2099-2108.*
3) The indentation at *TestFileSystemOperationsWithThreads.java L592-594* is
one of the recommended styles. Anyhow, I changed it to be a single line break.
All tests are passing with my tmarql3 endpoint:
Tests run: 775, Failures: 0, Errors: 0, Skipped: 155
> WASB: delete recursive should not fail if a file is deleted
> -----------------------------------------------------------
>
> Key: HADOOP-14769
> URL: https://issues.apache.org/jira/browse/HADOOP-14769
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: fs/azure
> Reporter: Thomas Marquardt
> Assignee: Thomas Marquardt
> Attachments: HADOOP-14769-001.patch, HADOOP-14769-002.patch
>
>
> FileSystem.delete(Path path) and delete(Path path, boolean recursive) return
> false if the path does not exist. The WASB implementation of recursive
> delete currently fails if one of the entries is deleted by an external agent
> while a recursive delete is in progress. For example, if you try to delete
> all of the files in a directory, which can be a very long process, and one of
> the files contained within is deleted by an external agent, the recursive
> directory delete operation will fail if it tries to delete that file and
> discovers that it does not exist. This is not desirable. A recursive
> directory delete operation should succeeed if the directory initially exists
> and when the operation completes, the directory and all of its entries do not
> exist.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
