Page MenuHomePhabricator

[services][backup] PullBackup 4/5 - blob stored item
ClosedPublic

Authored by bartek on Jan 11 2023, 3:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 1, 2:47 PM
Unknown Object (File)
Apr 13 2024, 2:57 PM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Unknown Object (File)
Apr 2 2024, 12:24 AM
Subscribers

Details

Summary

Second part of transport-business logic separation (see parent diff description). This diff abstracts both BackupItem and LogItem as some objects stored in blob service that can be sent to the client as PullBackupResponse.
Each stored object must have holder. Also, such object knows how to construct a response, what metadata to include in it and what size it is.

This trait will be used in the next diff to create a generic blob-service-download stream and avoid lots of code duplication.

Depends on D6242

Test Plan

This trait is tested altogether with other diffs for this endpoint

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Jan 11 2023, 3:56 PM
This revision is now accepted and ready to land.Jan 12 2023, 8:58 AM

Renamed fields_extra_size to metadata_size - this name is more accurate