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.

Sign up for GitHub

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

Jump to bottom

Document that broadcast capacity is lower bound #6042

Merged
merged 2 commits into from
Oct 3, 2023
Merged

Document that broadcast capacity is lower bound #6042

merged 2 commits into from
Oct 3, 2023

Conversation

Darksonn
Copy link
Contributor

@Darksonn Darksonn commented Oct 1, 2023

Closes: #6015

@Darksonn Darksonn added T-docs Topic: documentation A-tokio Area: The main tokio crate M-sync Module: tokio/sync labels Oct 1, 2023
@Darksonn Darksonn requested a review from hawkw October 1, 2023 20:42
@github-actions github-actions bot added the R-loom-sync Run loom sync tests on this PR label Oct 1, 2023
hawkw
hawkw reviewed Oct 2, 2023
View reviewed changes
tokio/src/sync/broadcast.rs Outdated Show resolved Hide resolved
@Darksonn Darksonn enabled auto-merge (squash) October 3, 2023 07:37
@Darksonn Darksonn merged commit eaba971 into master Oct 3, 2023
68 checks passed
@Darksonn Darksonn deleted the alice/broadcast-cap-doc branch October 3, 2023 07:58
@satakuma satakuma mentioned this pull request Oct 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hawkw hawkw hawkw approved these changes

Assignees
No one assigned
Labels
A-tokio Area: The main tokio crate M-sync Module: tokio/sync R-loom-sync Run loom sync tests on this PR T-docs Topic: documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Broadcast channel capacity is not well-documented
2 participants
@Darksonn @hawkw