Page MenuHomePhabricator
Differential D13618

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

Authored by bartek on Oct 4 2024, 11:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 10:53 AM
Unknown Object (File)
Thu, Dec 19, 10:53 AM
Unknown Object (File)
Thu, Dec 19, 10:53 AM
Unknown Object (File)
Thu, Dec 19, 10:53 AM
Unknown Object (File)
Thu, Dec 19, 10:53 AM
Unknown Object (File)
Sun, Nov 24, 4:10 PM
Unknown Object (File)
Sat, Nov 23, 2:40 AM
Unknown Object (File)
Nov 21 2024, 5:37 PM
View All 59 Files
Subscribers
ashoat
tomek

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
Lint Not Applicable
Unit
Tests Not Applicable

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
bartek updated this revision to Diff 44990.Oct 9 2024, 12:31 AM

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

PathSize
services/
blob/
src/
database/
40 lines
9 lines

Diff 45086

View Options

services/blob/src/database/client.rs

Loading...
View Options

services/blob/src/service.rs

Loading...