Page MenuHomeSoftware Heritage

Add support for slices in Storage.content_get (T1446)
AbandonedPublic

Authored by vlorentz on Apr 5 2019, 11:54 PM.
This revision can not be accepted until the required legal agreements have been signed.

Details

Summary

Changes are to be made in "Storage.content_get" to support the slicing of the data of full blob returned to it.
To execute this , two new parameters are added as an argument to the function "content_get()" to specify the rand of bytes that the caller requires
and also to pass them to the obstorage. Same changes have been made to both the files "swh/storage/in_memory.py" and "swh/storage/storage.py"

Diff Detail

Repository
rDSTO Storage manager
Branch
compute_all_counters
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5205
Build 7028: tox-on-jenkinsJenkins
Build 7027: arc lint + arc unit

Event Timeline

The parameter in the function (i.e. start and end) were not specified in earlier version, so correcting it.

correction made based on console result.

Thanks for the contribution but there is still some actions required from your side in order for the Software Heritage developers to review your changes:

Hi,

The task you are trying to solve (T1446, support of slices in swh-storage) depends on a much harder task (T1447, support of slices in swh-objstorage), so you shouldn't try to fix it now.

Sorry for tagging it as "easy hack", I underestimated the complexity of T1447.

@vloretz thank you so much and I shall take note of that. Does that mean
that my submission wouldn’t be revised and the patch wouldn’t be submitted
? Also, please let me know if I could be able to contribute to any other
hacks related to this hack.

  • in-memory storage: compute all missing counters
ardumont added inline comments.
.coverage.paris.28594.210279
2

For your information, those files (probably generated by tox or pytest) should not be committed.

This revision now requires changes to proceed.May 6 2019, 11:42 AM
vlorentz abandoned this revision.
vlorentz edited reviewers, added: Sowmya; removed: vlorentz.

Content Hidden

The content of this revision is hidden until the author has signed all of the required legal agreements.