Page MenuHomePhabricator

[blob-service] Implement holder management with new db
ClosedPublic

Authored by bartek on Jul 9 2023, 2:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 1:20 PM
Unknown Object (File)
Thu, Nov 7, 12:08 PM
Unknown Object (File)
Fri, Nov 1, 6:07 PM
Unknown Object (File)
Fri, Nov 1, 6:07 PM
Unknown Object (File)
Fri, Nov 1, 6:07 PM
Unknown Object (File)
Fri, Nov 1, 6:06 PM
Unknown Object (File)
Fri, Nov 1, 5:45 PM
Unknown Object (File)
Sun, Oct 27, 4:35 PM
Subscribers

Details

Summary

Part of ENG-4269.

Mostly copied logic from http::handlers::blob::assign_holder_handler and http::handlers::blob::remove_holder_handler - but now with using the new database methods

Partially addresses the TODO from http::handlers::blob::remove_holder_handler - auto deletion of orphaned blobs can now be disabled

Depends on D8455.

Test Plan

Tested later in the stack

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek edited the test plan for this revision. (Show Details)
bartek published this revision for review.Jul 10 2023, 12:44 AM

rust LGTM, would like for someone else to take a look

This revision is now accepted and ready to land.Jul 17 2023, 3:30 AM