-
Notifications
You must be signed in to change notification settings - Fork 141
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
Fix/message timestamp buffer #1831
Fix/message timestamp buffer #1831
Conversation
acda48b
to
9cad521
Compare
ee05257
to
6e5cdcf
Compare
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
Closes #1828.
This PR turns on
Messge
timestamp filtering by default. AnyMessage
with its timestamp differing too much from a node's local time will not be processed.Two preliminary tests were performed on 9c-main with no adverse effect; tested with
SwarmOptions.MessageTimestampBuffer
set to 60 seconds and 5 seconds.