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

congestion: decrease backpressure sensibility #11651

Merged
merged 3 commits into from
Jun 24, 2024

Conversation

jakmeier
Copy link
Contributor

With new limits on outgoing receipts, based on their size, the original parameters for backpressure on outgoing congestion are no longer ideal. Small numbers of receipts in the outgoing buffer will be a common case and should not cause backpressure.

With new limits on outgoing receipts, based on their size,
the original parameters for backpressure on outgoing
congestion are no longer ideal. Small numbers of receipts
in the outgoing buffer will be a common case and should
not cause backpressure.
@jakmeier jakmeier requested a review from a team as a code owner June 22, 2024 19:28
@jakmeier
Copy link
Contributor Author

Changing parameters as discussed here: https://near.zulipchat.com/#narrow/stream/295558-core/topic/outgoing.20receipts.20size.20limit/near/445084113

I created a new protocol version (82) guessing that we need it for a clean deployment on statlessnet. I'd also be happy to do the changes in version 80 if that's possible, just let me know.

Copy link

codecov bot commented Jun 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.65%. Comparing base (71d35d9) to head (68ae380).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11651      +/-   ##
==========================================
- Coverage   71.66%   71.65%   -0.02%     
==========================================
  Files         787      787              
  Lines      161063   161063              
  Branches   161063   161063              
==========================================
- Hits       115431   115404      -27     
- Misses      40584    40608      +24     
- Partials     5048     5051       +3     
Flag Coverage Δ
backward-compatibility 0.23% <ø> (ø)
db-migration 0.23% <ø> (ø)
genesis-check 1.36% <ø> (ø)
integration-tests 37.81% <ø> (-0.02%) ⬇️
linux 69.03% <ø> (-0.02%) ⬇️
linux-nightly 71.14% <ø> (-0.02%) ⬇️
macos 52.62% <ø> (+<0.01%) ⬆️
pytests 1.59% <ø> (ø)
sanity-checks 1.39% <ø> (ø)
unittests 66.22% <ø> (-0.02%) ⬇️
upgradability 0.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@bowenwang1996 bowenwang1996 left a comment

Choose a reason for hiding this comment

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

@wacban do we need to reorder protocol versions again after this PR is merged?

@wacban
Copy link
Contributor

wacban commented Jun 24, 2024

@wacban do we need to reorder protocol versions again after this PR is merged?

Yes, but let me see if we can squeeze it in to statelessnet today before it's launched.

@jakmeier
Copy link
Contributor Author

since we can squeeze it in before the network reset, I moved the changes to version 80 which keeps things simpler

Copy link
Contributor

@wacban wacban left a comment

Choose a reason for hiding this comment

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

LGTM

@jakmeier jakmeier added this pull request to the merge queue Jun 24, 2024
Merged via the queue into near:master with commit 74ac5fe Jun 24, 2024
30 checks passed
@jakmeier jakmeier deleted the cc-outgoing-buffer-increase branch June 24, 2024 11:37
tayfunelmas pushed a commit that referenced this pull request Jun 24, 2024
With new limits on outgoing receipts, based on their size, the original
parameters for backpressure on outgoing congestion are no longer ideal.
Small numbers of receipts in the outgoing buffer will be a common case
and should not cause backpressure.
telezhnaya pushed a commit that referenced this pull request Jun 24, 2024
With new limits on outgoing receipts, based on their size, the original
parameters for backpressure on outgoing congestion are no longer ideal.
Small numbers of receipts in the outgoing buffer will be a common case
and should not cause backpressure.
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