Skip to content

Conversation

@siddharthnohria
Copy link

No description provided.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 12, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@siddharthnohria siddharthnohria requested a review from ctiller May 14, 2025 11:34
* Status: Draft
* Implemented in: C++
* Last updated: 2025-05-12
* Discussion at: <google group thread> (filled after thread exists)
Copy link
Member

Choose a reason for hiding this comment

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

Please start the groups thread so that the 2wk timer starts

Copy link
Author

Choose a reason for hiding this comment

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

Done.

@siddharthnohria siddharthnohria changed the title Proposal for max_outstanding_streams in ResourceQuota L123: Proposal for server wide max_outstanding_streams May 14, 2025
@siddharthnohria siddharthnohria changed the title L123: Proposal for server wide max_outstanding_streams L124: Proposal for server wide max_outstanding_streams May 14, 2025
per-connection limit set to deal with overload situations (with 1000's of clients)
might be sub-optimal in situations with only a small number of clients.

## Proposal
Copy link
Member

Choose a reason for hiding this comment

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

Please document the APIs that you are adding at both the C-core layer and the C++ layer.

Copy link
Author

Choose a reason for hiding this comment

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

Done. Let me know if it looks fine.

@markdroth markdroth changed the title L124: Proposal for server wide max_outstanding_streams L124: C-core: Server-wide max_outstanding_streams May 14, 2025
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