Page MenuHomeSoftware Heritage

journal: Migrate to latest storage.origin_visit_upsert/add api change
ClosedPublic

Authored by ardumont on Mar 13 2020, 3:47 PM.

Details

Summary

Related to D2813
Related to D2820

Test Plan

tox

Diff Detail

Repository
rDJNL Journal infrastructure
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11125
Build 16777: tox-on-jenkinsJenkins
Build 16776: arc lint + arc unit

Event Timeline

ardumont retitled this revision from Migrate to latest origin_visit_upsert api change to Migrate to latest storage.origin_visit_upsert api change.Mar 13 2020, 3:49 PM
ardumont retitled this revision from Migrate to latest storage.origin_visit_upsert api change to journal: Migrate to latest storage.origin_visit_upsert api change.Mar 14 2020, 11:17 AM
anlambert added a subscriber: anlambert.

Looks good to me.

This revision is now accepted and ready to land.Mar 16 2020, 11:45 AM
swh/journal/tests/test_write_replay.py
58 ↗(On Diff #10076)

I'd be great to be able to use the objects() generator here.
But we can't as it's generated without consistency with the other types yet (e.g. Origin and OriginVisit for one)

So that makes the test fail.

ardumont retitled this revision from journal: Migrate to latest storage.origin_visit_upsert api change to journal: Migrate to latest storage.origin_visit_upsert/add api change.Mar 16 2020, 3:14 PM
ardumont edited the summary of this revision. (Show Details)

I don't understand the issue. What do you mean by "without consistency"?

And why is the origin visit date missing at all?

I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

And why is the origin visit date missing at all?

no idea.

I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

You could evolve it

I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

You could evolve it

sure.

That's not an issue for landing this diff.
I made the comment so that we see that future improvment in our hypothesis strategies in the model module.