Page MenuHomeSoftware Heritage

Add support for using URL instead of ID in origin_visit_{add,update}.
ClosedPublic

Authored by vlorentz on Jun 6 2019, 5:04 PM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers
Summary

Depends on D1557.

Diff Detail

Repository
rDSTO Storage manager
Branch
origin_visit_add_from_url
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6083
Build 8375: tox-on-jenkinsJenkins
Build 8374: arc lint + arc unit

Event Timeline

vlorentz created this revision.Jun 6 2019, 5:04 PM
ardumont accepted this revision.Jun 7 2019, 2:10 PM
ardumont added a subscriber: ardumont.

Sounds good as well, just a small adaptation needed.

swh/storage/storage.py
1229

Add missing db=db, cur=cur parameters as in line 1198 (of this file)

This revision is now accepted and ready to land.Jun 7 2019, 2:10 PM
vlorentz updated this revision to Diff 5134.Jun 7 2019, 2:18 PM

Pass db+cur and deduplicate calls to origin_get.

vlorentz closed this revision.Jun 11 2019, 1:22 PM

Landed as 0386716680da9032ba01fc34266ba96097420f67