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
Store naming #1528
Store naming #1528
Conversation
581eabb
to
25d49c2
Compare
Update S3 and Azure library adapter configurations in helper.py Update storage library classes to include a uid() method Update AzureStorage class to include container_name_ member variable
@@ -300,6 +300,10 @@ bool do_key_exists_impl( | |||
} | |||
} //namespace detail | |||
|
|||
std::string AzureStorage::name() const { | |||
return fmt::format("azure_storage-{}/{}", root_folder_, container_name_); |
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.
Are these the right way round?
@@ -168,7 +168,8 @@ class Library { | |||
} | |||
|
|||
std::string name() { | |||
return library_path_.to_delim_path(); | |||
auto lib_name = storages_->name(); | |||
return lib_name; |
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.
Why not just return storages_->name();
?
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.
This way it is easier to debug (at least in VSCode) and shouldn't really make a difference in a release build
Reference Issues/PRs
What does this implement or fix?
This PR is aiming to provide a way to get the name/identifier of a given store.
This is what the identifier looks like now - s3_storage-us-east-1/test_bucket_1/local/lib-kOGPh-3-True-target-1