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

Revert "Revert "[chttp2] fix stream leak with queued flow control update and absence of writes (#30907)" (#30991)" #30992

Merged
merged 2 commits into from Sep 15, 2022

Conversation

apolcyn
Copy link
Contributor

@apolcyn apolcyn commented Sep 14, 2022

Rolls forward #30907 after fixing the grpc_iomgr_count_objects_for_testing API.

Tested this works internally.

Copy link
Member

@ctiller ctiller left a comment

Choose a reason for hiding this comment

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

Please use the 'import/copybara' check to do a test import

@apolcyn
Copy link
Contributor Author

apolcyn commented Sep 15, 2022

import/copybara passed, and flakes here are pre-existing

@apolcyn apolcyn enabled auto-merge (squash) September 15, 2022 01:02
@apolcyn apolcyn merged commit dfb432c into grpc:master Sep 15, 2022
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bloat/none imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: yes Indicates if PR needs to be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants