Skip to content

Conversation

@simon-lemay-unity
Copy link
Contributor

@simon-lemay-unity simon-lemay-unity commented Aug 12, 2025

Purpose of this PR

Backport of PR #3593.

Changelog

Didn't seem worthy of a changelog entry. Can add one if requested.

Documentation

No documentation changes necessary.

Testing & QA

No need to test anything. It's just a string change.

Backport

#3593

@simon-lemay-unity simon-lemay-unity requested a review from a team as a code owner August 12, 2025 17:01
@simon-lemay-unity simon-lemay-unity changed the title [Backport] fix: Correct error message about send queue being full fix: [Backport] Correct error message about send queue being full Aug 12, 2025
Copy link
Collaborator

@michalChrobot michalChrobot left a comment

Choose a reason for hiding this comment

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

Nice catch!

@simon-lemay-unity simon-lemay-unity merged commit a9a8868 into develop Aug 13, 2025
29 of 30 checks passed
@simon-lemay-unity simon-lemay-unity deleted the fix-1.x/queue-size-error branch August 13, 2025 14:15
NoelStephensUnity added a commit that referenced this pull request Aug 15, 2025
## Purpose of this PR

Error message you get when the UTP send queue is full would indicate to
increase the 'Max Send Queue Size' parameter. But it really should be
the 'Max Packet Queue Size'. The send queue size is not something users
should have to modify since we increase dynamically as needed. The
packet queue size however should be increased by users if needed since
we can't increase it dynamically (at least with current UTP).

### Changelog

Didn't seem worthy of a changelog entry. Can add one if requested.

## Testing & QA

No need to test anything. It's just a string change.

## Documentation

No documentation changes necessary.

## Backport

#3594

Co-authored-by: Unity Netcode CI <[email protected]>
Co-authored-by: Noel Stephens <[email protected]>
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.

3 participants