[services] Rust Integration - Backup - Rust - Fix backup utils
AbandonedPublic
Actions

Authored by • karol on Sep 9 2022, 7:07 AM.

Details

Reviewers

• jon
tomek
varun
• max

Summary

Depends on D5097

This fixes the incorrect code from D5072. This commit didn't ruin anything but produced redundant code.

Test Plan

Integration tests don't raise warning for the redundant code anymore

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• karol created this revision.Sep 9 2022, 7:07 AM

Herald added a reviewer: • jon. · View Herald TranscriptSep 9 2022, 7:07 AM

Herald added subscribers: • abosh, atul, tomek, ashoat. · View Herald Transcript

• karol edited the summary of this revision. (Show Details)Sep 9 2022, 7:09 AM

• karol edited the test plan for this revision. (Show Details)

• karol added reviewers: tomek, varun, • max.

• karol added a parent revision: D5097: [services] Backup - Blob Get Client - Cleanup.

• karol added a child revision: D5099: [services] Backup - Blob Get Client - Pass rust string instead of const char*.

Harbormaster completed remote builds in B12054: Diff 16554.Sep 9 2022, 7:17 AM

• karol requested review of this revision.Sep 9 2022, 7:17 AM

This fixes the incorrect code from D5072.

Why don't we just amend D5072? seems like D5072 is expanding the work done while looping through the values anyway.

This revision now requires changes to proceed.Sep 9 2022, 10:35 AM

Right 😅

Revision Contents
Changeset List

Path

Size

services/

commtest/

tests/

backup/

backup_utils.rs

4 lines

Diff 16554

View Options

services/commtest/tests/backup/backup_utils.rs

[services] Rust Integration - Backup - Rust - Fix backup utilsAbandonedPublicActions