fix: gossipsub seen_ttl should be in ms#6607
Conversation
|
Nice catch! Current discussions hint that the spec will probably define those number in Also worth noting that the mentioned changes haven't been merged in the specs yet, and are thus not part of a spec release. |
this needs to be fixed regardless of the unit specified in the spec since that's what implemented in once spec is finalized we should change in |
Performance Report✔️ no performance regression detected 🚀🚀 Significant benchmark improvement detected
Full benchmark results
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6607 +/- ##
=========================================
Coverage 61.61% 61.61%
=========================================
Files 556 556
Lines 58561 58567 +6
Branches 1859 1859
=========================================
+ Hits 36080 36084 +4
- Misses 22440 22442 +2
Partials 41 41 |
|
🎉 This PR is included in v1.18.0 🎉 |
Motivation
Fix #6569
Description
In lodestar,
seen_ttlshould be in ms, not secondCloses #6606