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 lazy bootstrap possible links and accounts #2315

Merged
merged 30 commits into from
Oct 11, 2019

Conversation

SergiySW
Copy link
Contributor

@SergiySW SergiySW commented Sep 24, 2019

Alternative to #2301 with less possibility to finish lazy bootstrap for fresh node in 1 run.

  • request bulk_pull for state blocks where previous block is legacy receive, open or change (doesn't contain balance in block)
  • request bulk_pull for most used destinations accounts
  • allow to drop lazy bulk_pulls after failed attempt for destiations or unknown state blocks

Best way to test: fresh sync on live network with --disable_legacy_bootstrap --disable_bootstrap_listener --disable_wallet_bootstrap (so only lazy bootstrap & live traffic are enabled)

@SergiySW SergiySW added this to the V20.0 milestone Sep 24, 2019
@SergiySW SergiySW self-assigned this Sep 24, 2019
Copy link
Contributor

@cryptocode cryptocode left a comment

Choose a reason for hiding this comment

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

Syncs fine on beta

nano/node/bootstrap/bootstrap.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@clemahieu clemahieu left a comment

Choose a reason for hiding this comment

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

Looks good once the other comments are resolved.

@SergiySW SergiySW merged commit 65da97f into nanocurrency:master Oct 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants