Page MenuHomePhabricator
Differential D5016

[services] Backup - Blob Get Client - Add terminate
ClosedPublic
Actions

Authored by • karol on Sep 1 2022, 4:09 AM.
Tags
None
Referenced Files
F5057781: D5016.id16475.diff
Wed, Mar 26, 1:29 AM
Unknown Object (File)
Wed, Mar 19, 2:10 PM
Unknown Object (File)
Mon, Mar 17, 9:07 PM
Unknown Object (File)
Sun, Mar 16, 4:21 PM
Unknown Object (File)
Sun, Mar 16, 12:23 AM
Unknown Object (File)
Tue, Mar 11, 6:21 AM
Unknown Object (File)
Mon, Mar 10, 7:29 PM
Unknown Object (File)
Sun, Mar 9, 6:50 AM
View All Files
Subscribers
• abosh
ashoat
atul
tomek

Details

Reviewers
• jon
tomek
varun
• max
Commits
rCOMM440114cd315c: [services] Backup - Blob Get Client - Add terminate
Summary

Depends on D5015

Adding terminate function to the blob get client.

Test Plan

backup builds

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• karol created this revision.Sep 1 2022, 4:09 AM
Herald added a reviewer: • jon. · View Herald TranscriptSep 1 2022, 4:09 AM
Herald added subscribers: • abosh, atul, tomek, ashoat. · View Herald Transcript
• karol edited the summary of this revision. (Show Details)Sep 1 2022, 4:25 AM
• karol edited the test plan for this revision. (Show Details)
• karol added reviewers: tomek, varun, • max.
• karol added a parent revision: D5015: [services] Backup - Blob Get Client - Add blocking read.
• karol added a child revision: D5017: [services] Backup - Use Blob client in c++ - Implement create new backup.
Harbormaster completed remote builds in B11773: Diff 16176.Sep 1 2022, 4:28 AM
• karol requested review of this revision.Sep 1 2022, 4:28 AM
• karol planned changes to this revision.Sep 2 2022, 7:16 AM
• karol updated this revision to Diff 16249.Sep 2 2022, 7:17 AM

update

Harbormaster completed remote builds in B11844: Diff 16249.Sep 2 2022, 7:27 AM
tomek accepted this revision.Sep 5 2022, 5:40 AM
This revision is now accepted and ready to land.Sep 5 2022, 5:40 AM
• karol updated this revision to Diff 16448.Sep 8 2022, 4:01 AM

rebase

This revision was landed with ongoing or failed builds.Sep 8 2022, 4:10 AM
Closed by commit rCOMM440114cd315c: [services] Backup - Blob Get Client - Add terminate (authored by • karol). · Explain Why
This revision was automatically updated to reflect the committed changes.
• karol added a commit: rCOMM440114cd315c: [services] Backup - Blob Get Client - Add terminate.
Harbormaster completed remote builds in B11993: Diff 16448.Sep 8 2022, 4:18 AM
ashoat added inline comments.Sep 15 2022, 6:39 AM
services/backup/blob_client/src/get_client.rs
151–163

We could have reduced indentation here by simply reversing conditions. See here

• karol added inline comments.Sep 15 2022, 6:46 AM
services/backup/blob_client/src/get_client.rs
151–163

Are you sure this is possible when dealing with if let? How exactly can we do this? We need the unpacked values inside of the condition.

ashoat added inline comments.Sep 16 2022, 7:14 AM
services/backup/blob_client/src/get_client.rs
151–163

Ah, you may be right. Sorry, I'm not so familiar with Rust...

Revision Contents
Changeset List

PathSize
services/
backup/
blob_client/
src/
26 lines

Diff 16475

View Options

services/backup/blob_client/src/get_client.rs

Show All 31 Lines.questionText {
color: var(--white-100); color: var(--white-100);
} }
.icon { .icon {
margin-left: 24px; margin-left: 24px;
} }
.answerContainer { .answerContainer {
max-height: 0; display: grid;
overflow: hidden; grid-template-rows: 0fr;
transition: grid-template-rows 500ms;
} }
.activeAnswerContainer { .activeAnswerContainer {
max-height: fit-content; grid-template-rows: 1fr;
} }
.answerText { .answerText {
color: var(--white-100); color: var(--white-100);
margin-top: 16px; margin-top: 16px;
overflow: hidden;
} }
.link { .link {
color: inherit; color: inherit;
text-decoration: underline; text-decoration: underline;
transition: color 150ms; transition: color 150ms;
} }
.link:hover { .link:hover {
color: var(--violet-dark-100); color: var(--violet-dark-100);
transition: color 150ms; transition: color 150ms;
} }