[comm-lib] Add remove_multiple_holders to Blob client
Needs ReviewPublic
Actions

Authored by bartek on Wed, Oct 9, 1:15 AM.

Details

Reviewers

kamil
varun
will

Summary

We're missing functions for batch holders endpoints in BlobServiceClient.
Started with the one that's needed for server-side holder cleanup

Depends on D13617

Test Plan

Started Blob service. Created a few holders.
Created a dummy test that calls this function.
Ran cargo test and verified that these holders were now removed from DDB

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Wed, Oct 9, 1:15 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Oct 9, 1:15 AM

bartek added a child revision: D13645: [comm-lib][blob-client] Extract error handling logic.Wed, Oct 9, 1:17 AM

bartek mentioned this in D13648: [comm-lib][blob-client] Add support for removing holders by tags.Wed, Oct 9, 1:26 AM

Harbormaster completed remote builds in B32124: Diff 44994.Wed, Oct 9, 1:33 AM

bartek published this revision for review.Wed, Oct 9, 1:55 AM

Revision Contents
Changeset List

Path

Size

shared/

comm-lib/

src/

blob/

client.rs

58 lines

types.rs

4 lines

Diff 44994

View Options

shared/comm-lib/src/blob/client.rs