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

Update the CDN #1578

Closed
wants to merge 4 commits into from
Closed

Update the CDN #1578

wants to merge 4 commits into from

Conversation

rob-maron
Copy link
Contributor

@rob-maron rob-maron commented Jun 10, 2024

  • Improves throughput ~4x for larger (1000 node) runs by reducing contention
  • Better logging
  • Moves allocation pool (limiter for the the amount of total outstanding bytes not processed) to be definable during runtime
  • Adds TCP+TLS support. On my local machine it's much faster (10x??) than QUIC for the scenario I tested, we may want to use it in the future. But it may be from lack of GSO support on MacOS. Will investigate to see if we should switch for next testnet
  • Adds latency buckets for tail latency of message received -> sent to everyone
    • Adds 30s sliding window tail latency metric

Unrelated changes:

  • Topics didn't like being updated out of band with HotShot, so I pulled them and and added static assertions that they are equal

Before merge:

  • Test changes in the CDN
  • Test changes in HotShot (demonstrable improvement)
  • Test changes in sequencer (staging)

@rob-maron
Copy link
Contributor Author

Closed in #1530

@rob-maron rob-maron closed this Jun 18, 2024
@Ancient123 Ancient123 deleted the rm/update-cdn-20240610 branch August 29, 2024 15:38
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.

2 participants