felipecrv commented on code in PR #39361:
URL: https://github.com/apache/arrow/pull/39361#discussion_r1436526722
##########
cpp/src/arrow/filesystem/azurefs.cc:
##########
@@ -1033,47 +1042,78 @@ class AzureFileSystem::Impl {
return info;
} catch (const Storage::StorageException& exception) {
if (exception.StatusCode == Http::HttpStatusCode::NotFound) {
- ARROW_ASSIGN_OR_RAISE(auto hns_support,
- HierarchicalNamespaceSupport(adlfs_client));
- if (hns_support == HNSSupport::kContainerNotFound ||
- hns_support == HNSSupport::kEnabled) {
- // If the hierarchical namespace is enabled, then the storage
account will
- // have explicit directories. Neither a file nor a directory was
found.
- info.set_type(FileType::NotFound);
+ return FileInfo{location.all, FileType::NotFound};
+ }
+ return ExceptionToStatus(
+ exception, "GetProperties for '", file_client.GetUrl(),
+ "' failed. GetFileInfo is unable to determine whether the path
exists.");
+ }
+ }
+
+ /// On flat namespace accounts there are no real directories. Directories are
+ /// implied by empty directory marker blobs with names ending in "/" or there
+ /// being blobs with names starting with the directory path.
+ ///
+ /// \pre location.path is not empty.
+ Result<FileInfo> GetFileInfo(const Blobs::BlobContainerClient&
container_client,
+ const AzureLocation& location) {
+ DCHECK(!location.path.empty());
+ Blobs::ListBlobsOptions options;
+ options.Prefix = internal::RemoveTrailingSlash(location.path);
+ options.PageSizeHint = 1;
+
+ try {
+ FileInfo info{location.all};
+ auto list_response = container_client.ListBlobsByHierarchy(kDelimiter,
options);
+ if (!list_response.BlobPrefixes.empty()) {
Review Comment:
Done.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
