Page MenuHomeSoftware Heritage

Serialize objstorage and database writes in content_add
ClosedPublic

Authored by olasd on Wed, Mar 18, 3:40 PM.

Details

Summary

Considering that the objstorage is idempotent, and that there's no rollback
feature, intermixing both "transactions" has no concrete benefit. This avoids
doing database transactions that are longer than needed.

Test Plan

tox tests are again unchanged

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

olasd created this revision.Wed, Mar 18, 3:40 PM
vlorentz accepted this revision.Wed, Mar 18, 3:56 PM
This revision is now accepted and ready to land.Wed, Mar 18, 3:56 PM
This revision was automatically updated to reflect the committed changes.