Page MenuHomeSoftware Heritage

snapshot_add: Check the origin visit exists.
ClosedPublic

Authored by vlorentz on Tue, Dec 4, 2:00 PM.

Details

Summary

Instead of failing silently (PG storage) or raising an un-documented
exception (in-mem storage).

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

vlorentz created this revision.Tue, Dec 4, 2:00 PM
anlambert accepted this revision.Tue, Dec 4, 2:26 PM
anlambert added inline comments.
swh/storage/db.py
585

s/ids/id/

This revision is now accepted and ready to land.Tue, Dec 4, 2:26 PM
vlorentz marked an inline comment as done.Tue, Dec 4, 3:55 PM
vlorentz added inline comments.
swh/storage/db.py
585

Nope: origin id + visit id

This revision was automatically updated to reflect the committed changes.