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

Beacon sync blocks queue mgmnt update #3108

Merged
merged 6 commits into from
Feb 25, 2025

Conversation

mjfh
Copy link
Contributor

@mjfh mjfh commented Feb 25, 2025

No description provided.

mjfh added 6 commits February 25, 2025 11:39
details
   When importing starts while peers are actively downloading, the
   system tends to loose download peers, most probably due to high
   system activity. By this behaviour, there will be extra waiting
   delays for finding and connecting to new download peers.

   For this reason, importing and downloading is serialised. Downloading
   does not take place while importing.
details
  When importing starts while peers are actively downloading, the
  system tends to loose download peers, most probably due to high
  system activity. By this behaviour, there will be extra waiting
  delays for finding and connecting to new download peers.

  For this reason, importing starts not before the staged blocks queue
  is filled up. The ramp up time to fill up is a fraction of the
  potential waiting time when losing peers.
why
  Better overall throughput for slightly increased memory usage
why
  In regular circumstances, the header queue has a few records most of
  time. Longer queues appear with unwieldy peers (bogus data, timeouts,
  etc.) if they happen to lock the lowest record so preventing from
  temporarily serialising the queue.
@mjfh mjfh force-pushed the Beacon-sync-blocks-queue-mgmnt-update branch from a83efea to aa874b1 Compare February 25, 2025 11:51
@mjfh mjfh merged commit 0f89c1f into master Feb 25, 2025
8 checks passed
@mjfh mjfh deleted the Beacon-sync-blocks-queue-mgmnt-update branch February 25, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant