Page MenuHomePhabricator
Differential D5114

[Services] Remove Arc usage
ClosedPublic
Actions

Authored by • jon on Sep 12 2022, 8:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 6:30 AM
Unknown Object (File)
Sat, May 18, 6:30 AM
Unknown Object (File)
Sat, May 18, 6:30 AM
Unknown Object (File)
Sat, May 18, 6:29 AM
Unknown Object (File)
Sat, May 18, 6:25 AM
Unknown Object (File)
Thu, May 2, 8:18 AM
Unknown Object (File)
Wed, May 1, 6:21 PM
Unknown Object (File)
Wed, May 1, 6:21 PM
View All 84 Files
Subscribers
• abosh
ashoat

Details

Reviewers
• max
• karol
varun
tomek
atul
Commits
rCOMM70ed8a7b9d35: [Services] Remove Arc usage
Summary

Arc allows for the release of resources to be delegated to the runtime.
However, we have a global singleton which will always be in scope, so we don't
have to worry about it needing to be dynamically released.

https://linear.app/comm/issue/ENG-1732

Test Plan

Backup CI passes

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• jon created this revision.Sep 12 2022, 8:05 AM
Herald added subscribers: • abosh, ashoat. · View Herald TranscriptSep 12 2022, 8:05 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Sep 12 2022, 8:09 AM
Harbormaster failed remote builds in B12087: Diff 16588!
• jon updated this revision to Diff 16589.Sep 12 2022, 8:12 AM

Remove other references to Arc

Harbormaster completed remote builds in B12088: Diff 16589.Sep 12 2022, 8:22 AM
• jon requested review of this revision.Sep 12 2022, 8:22 AM
atul resigned from this revision.Sep 12 2022, 10:26 AM

(Resigning since I don't have the requisite Rust knowledge to review this yet)

• max accepted this revision.Sep 12 2022, 11:14 AM

That makes sense! We don't need an arc in lazy_static here.

This revision is now accepted and ready to land.Sep 12 2022, 11:14 AM
Closed by commit rCOMM70ed8a7b9d35: [Services] Remove Arc usage (authored by • jon). · Explain WhySep 13 2022, 12:16 PM
This revision was automatically updated to reflect the committed changes.
• jon added a commit: rCOMM70ed8a7b9d35: [Services] Remove Arc usage.
• jon mentioned this in D5035: [services] Rust Integration - Backup - Rust - Multiple put clients.Sep 13 2022, 12:16 PM

Revision Contents
Changeset List

PathSize
services/
backup/
blob_client/
src/
10 lines
10 lines
6 lines

Diff 16634

View Options

services/backup/blob_client/src/get_client.rs

Loading...
View Options

services/backup/blob_client/src/put_client.rs

Loading...
View Options

services/backup/blob_client/src/tools.rs

Loading...