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

improve p2p host connection manager, remove quic transporters #4707

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

GheisMohammadi
Copy link
Contributor

Issue

This PR improves p2p host connection manager and removes QUIC transporters from p2p host.

@Frozen
Copy link
Contributor

Frozen commented Jul 6, 2024

WithSilencePeriod sets the silence period.
The connection manager will perform a cleanup once per silence period
if the number of connections surpasses the high watermark.

@Frozen
Copy link
Contributor

Frozen commented Jul 6, 2024

WithGracePeriod sets the grace period.
The grace period is the time a newly opened connection is given before it becomes
subject to pruning.

@Frozen
Copy link
Contributor

Frozen commented Jul 6, 2024

What is our current "high watermark"?

@Frozen Frozen changed the title improve p2p host connection manager, remove quic transporters improved p2p host connection manager, removed quic transporters Jul 6, 2024
@Frozen
Copy link
Contributor

Frozen commented Jul 6, 2024

You have to merge this one as well to pass tests.

@GheisMohammadi GheisMohammadi changed the base branch from main to main_hotfix July 8, 2024 07:04
@GheisMohammadi
Copy link
Contributor Author

What is our current "high watermark"?

192

@GheisMohammadi GheisMohammadi changed the title improved p2p host connection manager, removed quic transporters improve p2p host connection manager, remove quic transporters Jul 8, 2024
@ONECasey ONECasey merged commit 2f4a616 into main_hotfix Jul 11, 2024
4 checks passed
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.

4 participants