This commit uses URLs *instead of* IDs, not in addition to.
Supporting IDs should not be needed anymore.
Details
Diff Detail
- Repository
- rDSTO Storage manager
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Build is green
See https://jenkins.softwareheritage.org/job/DSTO/job/tox/651/ for more details.
I guess you only really need the hunk in the postgres storage? What is the in-memory storage change trying to achieve ?
swh/storage/in_memory.py | ||
---|---|---|
76 | the argument should probably be renamed content_and_origins | |
140 | contents_and_origins :P | |
143 | Surely that only works because we only add contents from a single origin at a time; after the filtering, skipped_content_missing and origins aren't the same length any more. You really need to pass the full content to skipped_content_missing, then do the content/origin splitting. Which, in addition to the double-zipping, makes me wonder if that's really the right way to go at all. | |
155–158 | Could you turn this into a for loop? This isn't very readable. | |
190 | content_and_origins? |
Build is green
See https://jenkins.softwareheritage.org/job/DSTO/job/tox/652/ for more details.