Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sd_hash param to claim_search #3511

Merged
merged 8 commits into from
Jan 5, 2022
Merged

Add sd_hash param to claim_search #3511

merged 8 commits into from
Jan 5, 2022

Conversation

shyba
Copy link
Member

@shyba shyba commented Dec 8, 2021

backwards-incompatible: For hub operators: After updating to this release you need to resync the elasticsearch volume

this enables reverse search content from DHT (which claim belongs to some announcement) and ES indexing of stream descriptor hashes

@shyba shyba requested a review from jackrobison December 8, 2021 07:29
@shyba shyba linked an issue Dec 14, 2021 that may be closed by this pull request
@shyba shyba force-pushed the search_by_sd_hash branch from c899977 to 80c85cd Compare December 14, 2021 21:45
@shyba shyba changed the title Search by stream descriptor hash or prefix (sd_hash) (wip:needs go hub changes) Search by stream descriptor hash or prefix (sd_hash) Dec 14, 2021
@shyba shyba marked this pull request as ready for review December 14, 2021 21:45
@coveralls
Copy link

coveralls commented Dec 20, 2021

Coverage Status

Coverage remained the same at 69.685% when pulling bcf668b on search_by_sd_hash into 8076000 on master.

@shyba shyba force-pushed the search_by_sd_hash branch from 3c3401a to eb2b919 Compare December 22, 2021 16:29
@shyba shyba merged commit 7df0230 into master Jan 5, 2022
@shyba shyba deleted the search_by_sd_hash branch January 5, 2022 20:10
@jackrobison jackrobison changed the title Search by stream descriptor hash or prefix (sd_hash) Add sd_hash param to claim_search Mar 4, 2022
@jackrobison jackrobison added type: new feature New functionality that does not exist yet area: discovery labels Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: discovery type: new feature New functionality that does not exist yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Measure local ratio of content on blockchain vs announced on DHT
3 participants