Skip to content

Faster on-demand (instead of batched) announcement of blob hashes#3563

Merged
shyba merged 1 commit intodht_bugfrom
new_hash_announcer
Feb 21, 2022
Merged

Faster on-demand (instead of batched) announcement of blob hashes#3563
shyba merged 1 commit intodht_bugfrom
new_hash_announcer

Conversation

@shyba
Copy link
Copy Markdown
Member

@shyba shyba commented Feb 9, 2022

Makes the hash announcer consume from a queue instead of just gathering batches (we were pausing for each gather, instead this gather consumers for a queue)

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 9, 2022

Coverage Status

Coverage remained the same at 70.176% when pulling 4ecd2cb on new_hash_announcer into bfaf1b0 on master.

@shyba shyba force-pushed the new_hash_announcer branch from bfda82b to 77daf6d Compare February 15, 2022 03:55
@shyba shyba force-pushed the new_hash_announcer branch from 77daf6d to 4ecd2cb Compare February 19, 2022 05:51
@shyba shyba requested a review from jackrobison February 19, 2022 05:51
@lbry-bot lbry-bot assigned jackrobison and unassigned shyba Feb 19, 2022
@shyba shyba marked this pull request as ready for review February 19, 2022 05:51
@shyba shyba changed the base branch from master to dht_bug February 21, 2022 01:26
@shyba shyba added area: DHT type: improvement Existing (or partially existing) functionality needs to be changed labels Feb 21, 2022
@shyba
Copy link
Copy Markdown
Member Author

shyba commented Feb 21, 2022

since it depends on dht_bug, I changed the base there and will merge to unify the PRs

@shyba shyba merged commit d8c3964 into dht_bug Feb 21, 2022
@shyba shyba deleted the new_hash_announcer branch February 21, 2022 01:30
@jackrobison jackrobison changed the title DHT: make hash announcer use consumers instead of gathering batches Faster on-demand (instead of batched) announcement of blob hashes Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: DHT type: improvement Existing (or partially existing) functionality needs to be changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants