Skip to content

Remove yamux_window_size from network config#7014

Merged
bkchr merged 4 commits intoparitytech:masterfrom
qiweiii:remove-yamux_window_size
Feb 17, 2025
Merged

Remove yamux_window_size from network config#7014
bkchr merged 4 commits intoparitytech:masterfrom
qiweiii:remove-yamux_window_size

Conversation

@qiweiii
Copy link
Contributor

@qiweiii qiweiii commented Dec 30, 2024

Description

resolve #6468

Checklist

  • My PR includes a detailed description as outlined in the "Description" and its two subsections above.
  • My PR follows the labeling requirements of this project (at minimum one label for T required)
    • External contributors: ask maintainers to put the right label on your PR.
  • I have made corresponding changes to the documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Dec 30, 2024

User @qiweiii, please sign the CLA here.

@bkchr bkchr requested a review from dmitry-markin December 30, 2024 10:30
@bkchr bkchr added the T0-node This PR/Issue is related to the topic “node”. label Dec 30, 2024
@bkchr
Copy link
Member

bkchr commented Dec 30, 2024

/cmd prdoc --bump major --audience node_dev

@bkchr bkchr requested a review from lexnv February 15, 2025 00:32
Copy link
Contributor

@lexnv lexnv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! Currently, litep2p uses yamux_window_size config. However, since the litep2p is not enabled by default, and we'll come soon with a release that removes the window size, we can safely merge this PR. Thanks for contributing 🙏

@bkchr bkchr added this pull request to the merge queue Feb 17, 2025
Merged via the queue into paritytech:master with commit 6b6dae8 Feb 17, 2025
208 of 217 checks passed
clangenb pushed a commit to clangenb/polkadot-sdk that referenced this pull request Feb 19, 2025
# Description

resolve paritytech#6468



# Checklist

* [x] My PR includes a detailed description as outlined in the
"Description" and its two subsections above.
* [x] My PR follows the [labeling requirements](

https://github.com/paritytech/polkadot-sdk/blob/master/docs/contributor/CONTRIBUTING.md#Process
) of this project (at minimum one label for `T` required)
* External contributors: ask maintainers to put the right label on your
PR.
* [x] I have made corresponding changes to the documentation (if
applicable)
* [x] I have added tests that prove my fix is effective or that my
feature works (if applicable)

---------

Co-authored-by: command-bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T0-node This PR/Issue is related to the topic “node”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

network: Remove CLI flag for yamux_window_size

3 participants