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
Revert #9830 #9931
Revert #9830 #9931
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.
#9830 was a spec requirement (ethereum/consensus-specs#2183) i dont think we should treat this as a simple revert, I think the better course of action will be:
1.) document this in an issue
2.) communicate this with other client teams
3.) get feedbacks
4.) then revert or find better alternatives
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.
Looks good to me. I have also observed a high increase in upload bandwidth since v2.0.3, but I did not verify that this PR resolves the issue yet.
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.
Looks like this changes did not make it to other client implementations. I will communicate this back to other client teams and the original issue
What type of PR is this?
Bug Fix
What does this PR do? Why is it needed?
While #9830 was following the consensus spec changes, it also quadrupled bandwidth requirements in certain situations for nodes. Beacon nodes now show significant increases in outbound traffic which was narrowed down to the large increases in
attestion_aggregate_proof
re-propagation.Which issues(s) does this PR fix?
N.A
Other notes for review