Page MenuHomePhabricator

[services] Blob - Add append holder implementation
AbandonedPublic

Authored by karol on Apr 19 2022, 1:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 13, 3:06 PM
Unknown Object (File)
Sun, Oct 13, 3:06 PM
Unknown Object (File)
Sun, Oct 13, 2:58 PM
Unknown Object (File)
Sat, Oct 5, 9:33 PM
Unknown Object (File)
Sep 19 2024, 11:19 PM
Unknown Object (File)
Sep 13 2024, 3:15 AM
Unknown Object (File)
Sep 3 2024, 10:47 AM
Unknown Object (File)
Sep 3 2024, 10:47 AM

Details

Reviewers
tomek
ashoat
Summary

Depends on D3772

Adding AppendHolder to the blob service implementation.

https://linear.app/comm/issue/ENG-672/create-addreferencetohashbyholder-in-the-blob-service

Test Plan
cd services
yarn run-blob-service

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

karol edited the test plan for this revision. (Show Details)
karol added a reviewer: tomek.

This diff depends on an ongoing proto discussion - this is really inefficient. We should put up a diff with .proto / arch review way before starting the implementation. The current pattern makes it expensive to do any significant changes.

Overall it looks ok!

This revision is now accepted and ready to land.Apr 21 2022, 2:03 AM
ashoat requested changes to this revision.Apr 21 2022, 1:35 PM

I'd rather not have this accepted until D3772 is... I fear @karol-bisztyga will land it as-is and then submit another follow-up diff to modify it after he gets changes requested on D3772

This revision now requires changes to proceed.Apr 21 2022, 1:35 PM

I'd rather not have this accepted until D3772 is... I fear @karol-bisztyga will land it as-is and then submit another follow-up diff to modify it after he gets changes requested on D3772

I wouldn't land a diff that has at least one unaccepted predecessor but I understand your concern.

This is a part of mistakenly added code, more details in https://phabricator.ashoat.com/D3772#105661.

Abandoning. Sorry for the inconvenience, thank you for the review.