Page MenuHomeSoftware Heritage

Move to streaming decompression API

Authored by olasd on Sep 30 2019, 10:30 PM.



This allows us to detect whether the (compressed) object got corrupted with
trailing data, which the plain decompress() api doesn't allow.

This will be useful in further diffs

Depends on D2044

Test Plan


Diff Detail

rDOBJS Object storage
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda requested changes to this revision.Oct 1 2019, 9:32 AM

I would expect a (series of) test(s) with statements like with pytest.raises(Error, match='Corrupt object') somewhere in this diff.

This revision now requires changes to proceed.Oct 1 2019, 9:32 AM
douardda accepted this revision.Oct 1 2019, 9:38 AM

Adding tests 2 or 3 diffs after is a bold move... UNFAIR :-)

This revision is now accepted and ready to land.Oct 1 2019, 9:38 AM
This revision was automatically updated to reflect the committed changes.