Tom-Newton commented on code in PR #46095:
URL: https://github.com/apache/arrow/pull/46095#discussion_r2055472127
##########
cpp/src/arrow/filesystem/azurefs.cc:
##########
@@ -1441,6 +1452,14 @@ Result<FileInfo> GetContainerPropsAsFileInfo(const
AzureLocation& location,
info.set_type(FileType::NotFound);
return info;
}
+
+ if (IsForbidden(exception)) {
+ // User delegated sas tokens don't allow for getting container
properties. Assume
+ // the container exists
+ info.set_type(FileType::Directory);
+ info.set_mtime(std::chrono::system_clock::now());
Review Comment:
Is it actually necessary to set a time? I can't remember. If not I think I
would rather have no time at all than a somewhat misleading timestamp.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
