Skip to content

Commit

Permalink
Revert "Revert "Revert "server: introduce ServerMetricRecorder API an…
Browse files Browse the repository at this point in the history
…d move per-call reporting from a C++ interceptor to a C-core filter (#32106)" (#32272)" (#32279)"

This reverts commit 4475e74.
  • Loading branch information
ctiller committed Feb 3, 2023
1 parent 1936015 commit 88bc633
Show file tree
Hide file tree
Showing 43 changed files with 634 additions and 1,194 deletions.
37 changes: 16 additions & 21 deletions BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -762,7 +762,6 @@ grpc_cc_library(
"grpc_base",
# standard plugins
"census",
"//src/core:grpc_backend_metric_filter",
"//src/core:grpc_deadline_filter",
"//src/core:grpc_client_authority_filter",
"//src/core:grpc_lb_policy_grpclb",
Expand Down Expand Up @@ -1824,7 +1823,6 @@ grpc_cc_library(
"grpc_security_base",
"grpc_service_config_impl",
"grpc_trace",
"grpcpp_backend_metric_recorder",
"grpcpp_call_metric_recorder",
"grpcpp_status",
"iomgr_timer",
Expand All @@ -1840,7 +1838,6 @@ grpc_cc_library(
"//src/core:error",
"//src/core:gpr_atm",
"//src/core:gpr_manual_constructor",
"//src/core:grpc_backend_metric_provider",
"//src/core:grpc_service_config",
"//src/core:grpc_transport_inproc",
"//src/core:json",
Expand Down Expand Up @@ -1893,7 +1890,6 @@ grpc_cc_library(
"grpc_service_config_impl",
"grpc_trace",
"grpc_unsecure",
"grpcpp_backend_metric_recorder",
"grpcpp_call_metric_recorder",
"grpcpp_status",
"iomgr_timer",
Expand All @@ -1905,7 +1901,6 @@ grpc_cc_library(
"//src/core:error",
"//src/core:gpr_atm",
"//src/core:gpr_manual_constructor",
"//src/core:grpc_backend_metric_provider",
"//src/core:grpc_insecure_credentials",
"//src/core:grpc_service_config",
"//src/core:grpc_transport_inproc",
Expand Down Expand Up @@ -1983,42 +1978,45 @@ grpc_cc_library(

grpc_cc_library(
name = "grpcpp_call_metric_recorder",
srcs = [
"src/cpp/server/orca/call_metric_recorder.cc",
],
external_deps = [
"absl/strings",
"absl/types:optional",
"upb_lib",
],
language = "c++",
public_hdrs = [
"include/grpcpp/ext/call_metric_recorder.h",
],
visibility = ["@grpc:public"],
deps = ["grpc++_public_hdrs"],
deps = [
"grpc++_public_hdrs",
"xds_orca_upb",
"//src/core:arena",
"//src/core:grpc_backend_metric_data",
],
)

grpc_cc_library(
name = "grpcpp_backend_metric_recorder",
name = "grpcpp_orca_interceptor",
srcs = [
"src/cpp/server/backend_metric_recorder.cc",
"src/cpp/server/orca/orca_interceptor.cc",
],
hdrs = [
"src/cpp/server/backend_metric_recorder.h",
"src/cpp/server/orca/orca_interceptor.h",
],
external_deps = [
"absl/base:core_headers",
"absl/strings",
"absl/types:optional",
],
language = "c++",
public_hdrs = [
"include/grpcpp/ext/server_metric_recorder.h",
],
visibility = ["@grpc:public"],
deps = [
"gpr",
"grpc++_public_hdrs",
"grpc_trace",
"grpc++",
"grpc_base",
"grpcpp_call_metric_recorder",
"//src/core:grpc_backend_metric_data",
"//src/core:grpc_backend_metric_provider",
],
)

Expand All @@ -2029,7 +2027,6 @@ grpc_cc_library(
],
external_deps = [
"absl/base:core_headers",
"absl/strings",
"absl/time",
"absl/types:optional",
"upb_lib",
Expand All @@ -2045,13 +2042,11 @@ grpc_cc_library(
"gpr",
"grpc++",
"grpc_base",
"grpcpp_backend_metric_recorder",
"protobuf_duration_upb",
"ref_counted_ptr",
"xds_orca_service_upb",
"xds_orca_upb",
"//src/core:default_event_engine",
"//src/core:grpc_backend_metric_data",
"//src/core:ref_counted",
"//src/core:time",
],
Expand Down
33 changes: 21 additions & 12 deletions CMakeLists.txt

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 88bc633

Please sign in to comment.