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

Sowmya created this revision.Apr 5 2019, 11:54 PM
Sowmya updated this revision to Diff 4361.Apr 6 2019, 12:03 AM

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

Sowmya updated this revision to Diff 4362.Apr 6 2019, 12:09 AM

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:

Sowmya updated this revision to Diff 4363.Apr 6 2019, 9:25 AM

Signed the legal docs

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.

Sowmya added a comment.Apr 6 2019, 4:42 PM

@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.

Sowmya updated this revision to Diff 4397.Apr 8 2019, 9:26 PM
  • 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.

vlorentz requested changes to this revision.May 6 2019, 11:42 AM
This revision now requires changes to proceed.May 6 2019, 11:42 AM
vlorentz commandeered this revision.Jul 1 2019, 5:20 PM
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.