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
Paths
| Differential D6386 Authored by bartek on Jan 25 2023, 2:28 AM.
Details 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
Event Timelinebartek held this revision as a draft. Herald added subscribers: atul, ashoat. · View Herald TranscriptJan 25 2023, 2:28 AM2023-01-25 02:28:07 (UTC-8) bartek added a child revision: D6387: [backup-service] Improve blob client logging.Jan 25 2023, 2:35 AM2023-01-25 02:35:51 (UTC-8) Harbormaster completed remote builds in B15626: Diff 21300.Jan 25 2023, 2:41 AM2023-01-25 02:41:06 (UTC-8) This revision is now accepted and ready to land.Jan 26 2023, 4:25 AM2023-01-26 04:25:57 (UTC-8) Closed by commit rCOMM20bce1cabf3a: [backup-service] Use shared 'BlobClient' in workers (authored by bartek). · Explain WhyJan 31 2023, 2:23 AM2023-01-31 02:23:21 (UTC-8) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 21657 services/backup/src/blob/downloader.rs
services/backup/src/blob/uploader.rs
services/backup/src/service/handlers/add_attachments.rs
services/backup/src/service/handlers/create_backup.rs
|