Page MenuHomePhabricator
Differential D6386

[backup-service] Use shared 'BlobClient' in workers
ClosedPublic
Actions

Authored by bartek on Jan 25 2023, 2:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 31, 3:05 AM
Unknown Object (File)
Wed, Jan 29, 12:19 AM
Unknown Object (File)
Mon, Jan 27, 5:29 AM
Unknown Object (File)
Mon, Jan 27, 5:29 AM
Unknown Object (File)
Mon, Jan 27, 5:29 AM
Unknown Object (File)
Jan 11 2025, 3:55 PM
Unknown Object (File)
Dec 27 2024, 1:40 PM
Unknown Object (File)
Dec 27 2024, 1:40 PM
View All Files
Subscribers
ashoat
atul

Details

Reviewers
varun
• jon
michal
tomek
Commits
rCOMM20bce1cabf3a: [backup-service] Use shared 'BlobClient' in workers
Summary

Part of https://linear.app/comm/issue/ENG-2688/backup-service-reuse-single-blob-client-instance

Modified BlobDownloader::start and BlobUploader::start to take BlobClient as an argument instead of creating its own instance.

Depends on D6385

Test Plan

Ran backup service integration and performance tests to confirm it works as expected

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
backup/
src/
blob/
13 lines
13 lines
service/
handlers/
8 lines
4 lines
2 lines
8 lines

Diff 21657

View Options

services/backup/src/blob/downloader.rs

Loading...
View Options

services/backup/src/blob/uploader.rs

Loading...
View Options

services/backup/src/service/handlers/add_attachments.rs

Loading...
View Options

services/backup/src/service/handlers/create_backup.rs

Loading...
View Options

services/backup/src/service/handlers/pull_backup.rs

Loading...
View Options

services/backup/src/service/handlers/send_log.rs

Loading...