-
-
Notifications
You must be signed in to change notification settings - Fork 311
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
feat: increase default target peers to 100 #6556
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
after 4 days, I found heap allocation on network thread spiked then get back to normal (see #6106), it's stable on the mainnet thread |
I've been running my mainnet node on 100 peers for almost 2 weeks now, effectiveness looks pretty good so far but I am also seeing a increase in heap memory over time. Might also be related to Deneb upgrade or a memory leak we introduced in the new release. Will collect some metrics with 50 peers now to see if it is related to increasing the peer count. |
ef4fdfc
to
800e20e
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6556 +/- ##
=========================================
Coverage 61.69% 61.69%
=========================================
Files 556 556
Lines 58820 58820
Branches 1887 1887
=========================================
Hits 36287 36287
Misses 22492 22492
Partials 41 41 |
there are a lot of users already increased the target peers to 100 and it's noted that the performance of nodes increases a lot with this so we decided to get this PR in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
feat: increase target peers to 100
🎉 This PR is included in v1.18.0 🎉 |
Motivation
Description
Closes #5799