-
Notifications
You must be signed in to change notification settings - Fork 490
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
config: update vFuture consensus and configuration for 5MB blocks #4273
Conversation
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.
good except maybe localTemplate.go needs a version[24] ?
maybe add network/wsPeer.go |
for comparison, this is what I've been running lately |
@brianolson config v23 was just introduced in the 320-rounds branch so I think it is OK to tack these new defaults on to it. |
Codecov Report
@@ Coverage Diff @@
## master #4273 +/- ##
==========================================
- Coverage 55.30% 55.27% -0.04%
==========================================
Files 395 395
Lines 50333 50334 +1
==========================================
- Hits 27838 27821 -17
- Misses 20114 20121 +7
- Partials 2381 2392 +11
Help us with your feedback. Take ten seconds to tell us how you rate us. |
…gorand#4273) * update vFuture consensus and configuration for 5MB blocks * remove obsoleted vFuture.AgreementFilterTimeoutPeriod0 = 4s * update maxMessageLength to 6MiB * update simulateAgreementWithLedgerFactory to support alternate FilterTimeout values since algorand#1896 * update VerifiedTranscationsCacheSize to 150k * change AgreementFilterTimeoutPeriod0 = 3.4s
Summary
This updates the vFuture consensus parameters for 5MB blocks, 3.5s filter timeout, and updates configuration defaults for the next release to use a larger assembly timeout and TX pool size.
Test Plan
Existing tests should pass.