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-37173: Rewrite getURIs to use getManyURIs #760
Conversation
Codecov ReportBase: 85.32% // Head: 85.32% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #760 +/- ##
==========================================
- Coverage 85.32% 85.32% -0.01%
==========================================
Files 260 260
Lines 34539 34560 +21
Branches 5813 5819 +6
==========================================
+ Hits 29472 29488 +16
- Misses 3823 3826 +3
- Partials 1244 1246 +2
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. |
This allows Datastore to assume that a DatasetRef is always hashable.
The DummyRegistry used for datastore tests needs to be able to rollback inserts if the transaction fails.
Previously only datastore records were queried.
This lets us reuse the logic in the other method and also ensures that the logic is the same in that we only look on disk in "trust" mode when there is no corresponding datastore record.
fb1efa7
to
2b4beec
Compare
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 a little uneasy about the OpaqueTableStorage
interface change, but I'd be content with more complete documentation to make it feel less like a carve-out for a particular testing class.
Otherwise looks good.
State that the transaction object is only needed if a database is not going to roll back automatically.
Checklist
doc/changes