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

Limit frontier request block count by default #3147

Merged

Conversation

SergiySW
Copy link
Contributor

Default limit is 1M blocks per frontier request, then after bulk pulls next request

Changes to RPC:

  • allow optional "account" for RPC "bootstrap_any" for starting attempt account
  • return "frontiers_age" & "last_account" for RPC "bootstrap_status" in legacy attempts

to reduce bandwidth usage and failure rate for millions of frontiers.

- 66.7% of bootstrap attempts after initial bootstrap and after node warm up after startup to request only frontier modified since 1 day ago (1 hour for beta)
- for long inactive node use last stored online weight timestamp minus 1 hour

Additionally disable bulk push for requests containing age option.
Allow start account
@SergiySW SergiySW added documentation This item indicates the need for or supplies updated or expanded documentation functionality quality improvements This item indicates the need for or supplies a better way to implement existing functionality rpc Changes related to Remote Procedure Calls labels Mar 15, 2021
@SergiySW SergiySW added this to the V22.0 milestone Mar 15, 2021
@SergiySW SergiySW requested a review from clemahieu March 15, 2021 19:28
@SergiySW SergiySW self-assigned this Mar 15, 2021
SergiySW added a commit to SergiySW/raiblocks that referenced this pull request Apr 1, 2021
nanocurrency#3147
Default limit is 1M blocks per frontier request, then after bulk pulls next request

Changes to RPC:
- allow optional "account" for RPC "bootstrap_any" for starting attempt account
- return "frontiers_age" & "last_account" for RPC "bootstrap_status" in legacy attempts
@SergiySW
Copy link
Contributor Author

SergiySW commented Apr 6, 2021

make configurable with lower limit 1024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This item indicates the need for or supplies updated or expanded documentation functionality quality improvements This item indicates the need for or supplies a better way to implement existing functionality rpc Changes related to Remote Procedure Calls
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants