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
DM-37995: Improve storage class override logic #791
Conversation
60edc7a
to
89534f8
Compare
Codecov ReportBase: 85.54% // Head: 85.54% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #791 +/- ##
=======================================
Coverage 85.54% 85.54%
=======================================
Files 266 266
Lines 35137 35186 +49
Branches 7377 7388 +11
=======================================
+ Hits 30057 30100 +43
- Misses 3765 3768 +3
- Partials 1315 1318 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
I'm not overly happy but this is the pragmatic solution :-)
4c9f5b1
to
738549a
Compare
In trusted mode the datastore has to use original registry storage class to search for artifacts. Because datastore has no access to registry a new method was added to Datastore class to specify a way to retrieve registry dataset type based on its name (dependency inversion). Butler instances use this new method to provide access to to dataset type mapping. In case of QBB, when registry does not exist this mapping has to be provided when constructing QBB.
Co-authored-by: Tim Jenness <tjenness@lsst.org>
738549a
to
d6265b5
Compare
In trusted mode the datastore has to use original registry storage class
to search for artifacts. Because datastore has no access to registry
a new method was added to Datastore class to specify a way to retrieve
registry dataset type based on its name (dependency inversion).
Butler instances use this new method to provide access to to dataset
type mapping. In case of QBB, when registry does not exist this mapping
has to be provided when constructing QBB.
Checklist
doc/changes