Skip to content

Conversation

@Giulio2002
Copy link
Contributor

@Giulio2002 Giulio2002 commented Sep 3, 2025

The fix was:

  • The previous peer selection policy was suboptimal so I just replaced it with random selection
  • be more aggressive with banning
  • increased the amount of grace peers
  • make sure 100% that EarliestSlotAvaiable=nil

This should actually improve the downloader's performance overall

@Giulio2002 Giulio2002 requested a review from domiwei September 3, 2025 21:52
@yperbasis yperbasis requested a review from anacrolix September 4, 2025 07:21
@Giulio2002 Giulio2002 enabled auto-merge (squash) September 4, 2025 13:24
@Giulio2002 Giulio2002 merged commit 131448e into main Sep 4, 2025
15 checks passed
@Giulio2002 Giulio2002 deleted the fix-gnosis-downloader branch September 4, 2025 13:54
Giulio2002 added a commit that referenced this pull request Sep 4, 2025
The fix was:
- The previous peer selection policy was suboptimal so I just replaced
it with random selection
- be more aggressive with banning
- increased the amount of grace peers
- make sure 100% that `EarliestSlotAvaiable=nil`

This should actually improve the downloader's performance overall

---------

Co-authored-by: Kewei <[email protected]>
bloxster pushed a commit that referenced this pull request Sep 9, 2025
The fix was:
- The previous peer selection policy was suboptimal so I just replaced
it with random selection
- be more aggressive with banning
- increased the amount of grace peers
- make sure 100% that `EarliestSlotAvaiable=nil`


This should actually improve the downloader's performance overall

---------

Co-authored-by: Kewei <[email protected]>
NazariiDenha pushed a commit that referenced this pull request Oct 24, 2025
The fix was:
- The previous peer selection policy was suboptimal so I just replaced
it with random selection
- be more aggressive with banning
- increased the amount of grace peers
- make sure 100% that `EarliestSlotAvaiable=nil`


This should actually improve the downloader's performance overall

---------

Co-authored-by: Kewei <[email protected]>
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.

3 participants