Page MenuHomePhabricator
Differential D5153

[Backup] Use only one tokio runtime
ClosedPublic
Actions

Authored by • jon on Sep 15 2022, 10:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 29, 1:10 AM
Unknown Object (File)
Fri, Dec 27, 8:38 AM
Unknown Object (File)
Fri, Dec 27, 8:38 AM
Unknown Object (File)
Fri, Dec 27, 8:38 AM
Unknown Object (File)
Wed, Dec 25, 10:23 PM
Unknown Object (File)
Wed, Dec 4, 6:42 PM
Unknown Object (File)
Wed, Dec 4, 6:40 PM
Unknown Object (File)
Wed, Dec 4, 6:33 PM
View All Files
Subscribers
• abosh
ashoat
tomek

Details

Reviewers
atul
varun
• karol
• max
Commits
rCOMM040488e9923b: [Backup] Use only one tokio runtime
Summary

We only need one runtime to do asynchronous calls. Using two or more
is an anti-pattern.

https://linear.app/comm/issue/ENG-1808
https://linear.app/comm/issue/ENG-1806

Test Plan
cd services/backup
rm -rf build && mkdir build && cd build && cmake .. && make -j

Diff Detail

Repository
rCOMM Comm
Branch
jonringer/remove-duplicate-runtimes
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
services/
backup/
blob_client/
src/
3 lines
9 lines
5 lines

Diff 16759

View Options

services/backup/blob_client/src/get_client.rs

Loading...
View Options

services/backup/blob_client/src/lib.rs

Loading...
View Options

services/backup/blob_client/src/put_client.rs

Loading...