Skip to content

Revert "Revert "Revert "Revert "server: introduce ServerMetricRecorde…#32301

Merged
markdroth merged 2 commits intogrpc:masterfrom
yousukseung:backend_metric_sampling_retry2
Feb 6, 2023
Merged

Revert "Revert "Revert "Revert "server: introduce ServerMetricRecorde…#32301
markdroth merged 2 commits intogrpc:masterfrom
yousukseung:backend_metric_sampling_retry2

Conversation

@yousukseung
Copy link
Contributor

…r API and move per-call reporting from a C++ interceptor to a C-core filter (#32106)" (#32272)" (#32279)" (#32293)"

This reverts commit 1f96069.

…r API and move per-call reporting from a C++ interceptor to a C-core filter (grpc#32106)" (grpc#32272)" (grpc#32279)" (grpc#32293)"

This reverts commit 1f96069.
@yousukseung yousukseung requested a review from markdroth February 6, 2023 18:55
@yousukseung yousukseung marked this pull request as ready for review February 6, 2023 18:55
@markdroth markdroth merged commit b98f527 into grpc:master Feb 6, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Feb 6, 2023
@yousukseung yousukseung deleted the backend_metric_sampling_retry2 branch February 22, 2023 19:16
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
grpc#32301)

* Revert "Revert "Revert "Revert "server: introduce ServerMetricRecorder API and move per-call reporting from a C++ interceptor to a C-core filter (grpc#32106)" (grpc#32272)" (grpc#32279)" (grpc#32293)"

This reverts commit 1f96069.

* Do not create CallMetricRecorder if call is null.
wanlin31 pushed a commit that referenced this pull request May 18, 2023
#32301)

* Revert "Revert "Revert "Revert "server: introduce ServerMetricRecorder API and move per-call reporting from a C++ interceptor to a C-core filter (#32106)" (#32272)" (#32279)" (#32293)"

This reverts commit 1f96069.

* Do not create CallMetricRecorder if call is null.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/medium imported Specifies if the PR has been imported to the internal repository lang/c++ lang/core lang/Python 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