[blob] Add DB method to query holders by prefix
ClosedPublic
Actions

Authored by bartek on Oct 4 2024, 11:46 AM.

Details

Reviewers

kamil
varun
will

Commits

rCOMM4762e1c22626: [blob] Add DB method to query holders by prefix

Summary

Part of ENG-9367.
Added DB function that queries the holder-tag index and returns found blobHash-holder pairs.

Depends on D13617

Test Plan

Tested together with the next diff

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Oct 4 2024, 11:46 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 4 2024, 11:46 AM

bartek added a child revision: D13619: [blob] Add endpoint to query holders by prefix.Oct 4 2024, 11:48 AM

Harbormaster completed remote builds in B32080: Diff 44913.Oct 4 2024, 12:05 PM

Rebase

Harbormaster completed remote builds in B32120: Diff 44990.Oct 9 2024, 12:52 AM

bartek added a child revision: D13647: [blob] Add support for removing holders by tags.Oct 9 2024, 1:22 AM

bartek published this revision for review.Oct 9 2024, 1:54 AM

kamil accepted this revision.Oct 10 2024, 1:58 AM

kamil added inline comments.

services/blob/src/database/client.rs
264 ↗	(On Diff #44990)	I think `tag` could be a better name

This revision is now accepted and ready to land.Oct 10 2024, 1:58 AM

Closed by commit rCOMM4762e1c22626: [blob] Add DB method to query holders by prefix (authored by bartek). · Explain WhyOct 10 2024, 2:06 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM4762e1c22626: [blob] Add DB method to query holders by prefix.

Revision Contents
Changeset List

Path

Size

services/

blob/

src/

database/

client.rs

37 lines

service.rs

9 lines

Diff 44913

View Options

services/blob/src/database/client.rs