Page MenuHomeSoftware Heritage

Use the in-mem storage.
ClosedPublic

Authored by vlorentz on Wed, Dec 5, 2:05 PM.

Details

Summary

Diff Detail

Repository
rDLDTAR Tarball Loader
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.Wed, Dec 5, 2:05 PM
ardumont added inline comments.
swh/loader/tar/tests/test_loader.py
109

why do you have to flush explicitely?

The loader-core's current behavior is supposed to do it.
(for now loader-tar -inherits-> loader-dir -inherits-> loader-core, for now because i'd like to simplify this inheritance chain, never had a chance to ;)

vlorentz marked an inline comment as done.Wed, Dec 5, 2:15 PM
vlorentz added inline comments.
swh/loader/tar/tests/test_loader.py
109

I don't. I was trying stuff and forgot to clean this

vlorentz updated this revision to Diff 2453.Wed, Dec 5, 2:15 PM
  • Remove useless call.
ardumont accepted this revision.Wed, Dec 5, 2:18 PM
This revision is now accepted and ready to land.Wed, Dec 5, 2:18 PM
vlorentz updated this revision to Diff 2456.Wed, Dec 5, 2:21 PM
  • squash
This revision was automatically updated to reflect the committed changes.