Skip to content

[stats] Add a test for IncrementHttp2MetadataSize(0)#32896

Merged
ctiller merged 1 commit intogrpc:masterfrom
ctiller:stats-0
Apr 19, 2023
Merged

[stats] Add a test for IncrementHttp2MetadataSize(0)#32896
ctiller merged 1 commit intogrpc:masterfrom
ctiller:stats-0

Conversation

@ctiller
Copy link
Copy Markdown
Member

@ctiller ctiller commented Apr 19, 2023

This test proves that global_stats.IncrementHttp2MetadataSize(0) works

@ctiller ctiller requested a review from nanahpang April 19, 2023 14:45
@ctiller ctiller enabled auto-merge (squash) April 19, 2023 14:48
@ctiller ctiller added the release notes: no Indicates if PR should not be in release notes label Apr 19, 2023
@ctiller ctiller merged commit 9128604 into grpc:master Apr 19, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Apr 19, 2023
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
This test proves that `global_stats.IncrementHttp2MetadataSize(0)` works

<!--

If you know who should review your pull request, please assign it to
that
person, otherwise the pull request would get assigned randomly.

If your pull request is for a specific language, please add the
appropriate
lang label.

-->
wanlin31 pushed a commit that referenced this pull request May 18, 2023
This test proves that `global_stats.IncrementHttp2MetadataSize(0)` works

<!--

If you know who should review your pull request, please assign it to
that
person, otherwise the pull request would get assigned randomly.

If your pull request is for a specific language, please add the
appropriate
lang label.

-->
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: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants