Skip to content

Only announce sd blobs, stop announcing head blobs#3565

Merged
shyba merged 4 commits intomasterfrom
stop_announcing_head_blob
Feb 21, 2022
Merged

Only announce sd blobs, stop announcing head blobs#3565
shyba merged 4 commits intomasterfrom
stop_announcing_head_blob

Conversation

@shyba
Copy link
Copy Markdown
Member

@shyba shyba commented Feb 9, 2022

backwards-incompatible: For desktop / lbry-sdk users: this release includes a migration that increasing the database version. This usually means you can't downgrade without a backup of lbrynet.sqlite.

This should make 1 announcement per stream instead of 2.

  • stop announcing the head blob
  • stop searching for it
  • migration script to set all head blobs to should_announce=false

part of #3527 but probably doesn't close it (faster hash announcer and tracker to go)

@shyba shyba added type: improvement Existing (or partially existing) functionality needs to be changed area: DHT area: blobs/files labels Feb 9, 2022
@coveralls
Copy link
Copy Markdown

coveralls commented Feb 9, 2022

Coverage Status

Coverage remained the same at 69.7% when pulling 3ee3e7f on stop_announcing_head_blob into 43ac333 on master.

@shyba shyba marked this pull request as ready for review February 9, 2022 03:08
@shyba shyba requested a review from jackrobison February 9, 2022 03:08
@lbry-bot lbry-bot assigned jackrobison and unassigned shyba Feb 9, 2022
@shyba shyba force-pushed the stop_announcing_head_blob branch from bf82e5c to 43dd675 Compare February 9, 2022 03:12
from .migrate13to14 import do_migration
elif current == 14:
from .migrate14to15 import do_migration
elif current == 15:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be documented as a breaking change, users will only be able to downgrade by manually editing the version file

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

@lbry-bot lbry-bot assigned shyba and unassigned shyba Feb 16, 2022
@shyba shyba force-pushed the stop_announcing_head_blob branch from 43dd675 to 3ee3e7f Compare February 21, 2022 01:25
@shyba shyba merged commit dd5b9ca into master Feb 21, 2022
@shyba shyba deleted the stop_announcing_head_blob branch February 21, 2022 01:33
@jackrobison jackrobison changed the title DHT: stop announcing the head blob Only announce sd blobs, stop announcing head blobs Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: blobs/files 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