Skip to content
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

Handle aggregates and contributions concurrently #21

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

eth2353
Copy link
Member

@eth2353 eth2353 commented Sep 24, 2024

Before this PR, this is how attestation aggregates were done:
(same applies to sync committee contributions)

  1. Get aggregate attestations for all aggregator duties
  2. Sign all the aggregate attestations
  3. Publish all the aggregate attestations

-> If one of the aggregates took a bit longer to retrieve, the rest of them also waited for it, even though they could have already been signed and published.

With this PR, once any aggregate attestation is available, it is signed and published in a separate, independent task.

@eth2353 eth2353 merged commit 1269253 into master Sep 24, 2024
2 checks passed
@eth2353 eth2353 deleted the aggregation-performance branch September 24, 2024 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant