Closed arnetheduck closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.56%. Comparing base (
08a48fa
) to head (fac3fb4
).
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.83%. Comparing base (
d1e51be
) to head (e469c14
). Report is 1 commits behind head on unstable.
Thanks for that, looks much better now anyway IMO.
The way the code is written, a new Opt instance is created for every RPC message processed by gossipsub - this involves copying the full message even though only the size is needed for rate limiting purposes - in Nimbus, this becomes a performance/memory allocation hotspot.