-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add explicit model control test for Azure storage #3002
Conversation
42135eb
to
78cca68
Compare
- for cases where errno is set to 0 but blob_property.valid() = false
@@ -916,7 +916,7 @@ ASFileSystem::FileModificationTime(const std::string& path, int64_t* mtime_ns) | |||
RETURN_IF_ERROR(ParsePath(path, &container, &object_path)); | |||
|
|||
auto blobProperty = bc.get_blob_property(container, object_path); | |||
if (!blobProperty.valid()) { | |||
if (errno != 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very strange for a C++ API to use errno...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is what I thought as well. Additionally there are some cases where the valid is set to false even when errno is set to 0. While I was not able to reproduce such a case this is what the caused the error that the user was seeing in issue 2966
* Add explicit model control test for Azure storage * Update copyright * Fix incorrectly detected failure when getting blob property - for cases where errno is set to 0 but blob_property.valid() = false * Update copyright * Improve verbose error logging for azure storage FS
* Add explicit model control test for Azure storage * Update copyright * Fix incorrectly detected failure when getting blob property - for cases where errno is set to 0 but blob_property.valid() = false * Update copyright * Improve verbose error logging for azure storage FS
Fix case errno is 0 but the blob_property.valid() returns false. It appears errno is the correct way to check for errors based on the azure sdk sample.