Page MenuHomePhabricator

[backup-service] Propagate 'BlobClient' to handlers
ClosedPublic

Authored by bartek on Jan 25 2023, 2:23 AM.
Tags
None
Referenced Files
F3504658: D6385.id21299.diff
Fri, Dec 20, 9:54 AM
F3504651: D6385.id21656.diff
Fri, Dec 20, 9:53 AM
F3502639: D6385.diff
Fri, Dec 20, 4:31 AM
Unknown Object (File)
Wed, Nov 20, 4:07 PM
Unknown Object (File)
Nov 10 2024, 2:00 PM
Unknown Object (File)
Nov 9 2024, 10:25 AM
Unknown Object (File)
Nov 9 2024, 6:37 AM
Unknown Object (File)
Nov 9 2024, 6:28 AM
Subscribers

Details

Summary

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

Citing https://phab.comm.dev/D6167?id=20696#inline-41664 and Tonic docs - the Channel is encouraged to be cloned and so the BlobClient instance is.
I propagated it to all places where it would be needed, the same way as DynamoDB DatabaseClient. It's not yet used, though.

Depends on D6384

Test Plan

Backup service compiles.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage