Page MenuHomePhabricator
Differential D6383

[backup-service] Rename 'PutClient' to 'BlobUploader'
ClosedPublic
Actions

Authored by bartek on Jan 25 2023, 2:13 AM.
Tags
None
Referenced Files
F4128127: D6383.diff
Tue, Feb 18, 8:39 AM
Unknown Object (File)
Wed, Jan 29, 12:19 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:28 AM
Unknown Object (File)
Jan 11 2025, 3:55 PM
Unknown Object (File)
Jan 10 2025, 3:25 PM
Unknown Object (File)
Dec 27 2024, 1:40 PM
View All Files
Subscribers
ashoat
atul

Details

Reviewers
varun
• jon
michal
tomek
Commits
rCOMM75bef826f30b: [backup-service] Rename 'PutClient' to 'BlobUploader'
Summary

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

Renamed PutClient to BlobUploader, because this name is more accurate - these instances are not clients, but rather single RPC call instances. Also, old name could easily be confused with real Blob Client instance, introduced in further diffs.

Depends on D6382

Test Plan

Backup service compiles and starts.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
backup/
src/
blob/
4 lines
uploader.rs
put_client.rs
28 lines
service/
handlers/
8 lines
20 lines
20 lines

Diff 21654

View Options

services/backup/src/blob/mod.rs

Loading...
View Options

services/backup/src/blob/put_client.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/send_log.rs

Loading...