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

fix(perf): replace rust-libp2p-quinn/ with rust-libp2p/master #246

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented Aug 1, 2023

With libp2p/rust-libp2p#3454 merged we can now test the new rust-libp2p QUIC implementation based on upstream quinn directly from rust-libp2p master.

This commit does the following in libp2p/test-plans:

  1. Remove the perf/impl/rust-libp2p-quinn implementation.
  2. Introduce the perf/impl/rust-libp2p/master version of the rust-libp2p implementation.

We can promote the latter to a proper version on the next rust-libp2p release.

With libp2p/rust-libp2p#3454 merged we can now test the
new rust-libp2p QUIC implementation based on upstream quinn directly from
rust-libp2p `master`.

This commit does the following in libp2p/test-plans:

1. Remove the `perf/impl/rust-libp2p-quinn` implementation.
2. Introduce the `perf/impl/rust-libp2p/master` version of the rust-libp2p implementation.

We can promote the latter to a proper version on the next rust-libp2p release.
@mxinden
Copy link
Member Author

mxinden commented Aug 1, 2023

@mxinden
Copy link
Member Author

mxinden commented Aug 1, 2023

One can now compare rust-libp2p/v0.52 with rust-libp2p/master at the bottom of the dashboard.

https://observablehq.com/@libp2p-workspace/performance-dashboard?branch=perf-rust-libp2p-quinn#branch

@mxinden
Copy link
Member Author

mxinden commented Aug 1, 2023

Merging here since it only touches perf/. Happy to do follow-up changes.

//CC @thomaseizinger

@mxinden mxinden merged commit ffc7ae0 into master Aug 1, 2023
@mxinden mxinden deleted the perf-rust-libp2p-quinn branch August 1, 2023 14:40
@mxinden
Copy link
Member Author

mxinden commented Aug 1, 2023

One can now compare rust-libp2p/v0.52 with rust-libp2p/master at the bottom of the dashboard.

https://observablehq.com/@libp2p-workspace/performance-dashboard?branch=perf-rust-libp2p-quinn#branch

Also //CC @kpp in case you want to see the fruits of your work.

mxinden added a commit that referenced this pull request Aug 24, 2023
Implementation has been removed with #246.
mxinden added a commit that referenced this pull request Aug 24, 2023
Implementation has been removed with #246.
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