grpc 1.63.0 → 1.64.0.pre1

data/src/core/lib/channel/call_finalization.h

@@ -15,10 +15,10 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CALL_FINALIZATION_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CALL_FINALIZATION_H
 
-#include <grpc/support/port_platform.h>
-
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/promise/context.h"
 #include "src/core/lib/resource_quota/arena.h"
 #include "src/core/lib/transport/call_final_info.h"

data/src/core/lib/channel/call_tracer.cc

@@ -16,15 +16,16 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/call_tracer.h"
 
 #include <memory>
 #include <utility>
 #include <vector>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/channel/tcp_tracer.h"
 #include "src/core/lib/promise/context.h"
@@ -77,7 +78,7 @@ class DelegatingClientCallTracer : public ClientCallTracer {
     explicit DelegatingClientCallAttemptTracer(
         std::vector<CallAttemptTracer*> tracers)
         : tracers_(std::move(tracers)) {
-      GPR_DEBUG_ASSERT(!tracers_.empty());
+      DCHECK(!tracers_.empty());
     }
     ~DelegatingClientCallAttemptTracer() override {}
     void RecordSendInitialMetadata(
@@ -177,7 +178,7 @@ class DelegatingClientCallTracer : public ClientCallTracer {
     attempt_tracers.reserve(tracers_.size());
     for (auto* tracer : tracers_) {
       auto* attempt_tracer = tracer->StartNewAttempt(is_transparent_retry);
-      GPR_DEBUG_ASSERT(attempt_tracer != nullptr);
+      DCHECK_NE(attempt_tracer, nullptr);
       attempt_tracers.push_back(attempt_tracer);
     }
     return GetContext<Arena>()->ManagedNew<DelegatingClientCallAttemptTracer>(
@@ -329,9 +330,8 @@ void AddClientCallTracerToContext(grpc_call_context_element* call_context,
 
 void AddServerCallTracerToContext(grpc_call_context_element* call_context,
                                   ServerCallTracer* tracer) {
-  GPR_DEBUG_ASSERT(
-      call_context[GRPC_CONTEXT_CALL_TRACER].value ==
-      call_context[GRPC_CONTEXT_CALL_TRACER_ANNOTATION_INTERFACE].value);
+  DCHECK(call_context[GRPC_CONTEXT_CALL_TRACER].value ==
+         call_context[GRPC_CONTEXT_CALL_TRACER_ANNOTATION_INTERFACE].value);
   if (call_context[GRPC_CONTEXT_CALL_TRACER_ANNOTATION_INTERFACE].value ==
       nullptr) {
     // This is the first call tracer. Set it directly.

data/src/core/lib/channel/call_tracer.h

@@ -19,8 +19,6 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CALL_TRACER_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CALL_TRACER_H
 
-#include <grpc/support/port_platform.h>
-
 #include <memory>
 #include <string>
 
@@ -28,6 +26,7 @@
 #include "absl/strings/string_view.h"
 #include "absl/types/optional.h"
 
+#include <grpc/support/port_platform.h>
 #include <grpc/support/time.h>
 
 #include "src/core/lib/channel/channel_args.h"

data/src/core/lib/channel/channel_args.cc

@@ -16,8 +16,6 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_args.h"
 
 #include <limits.h>
@@ -30,6 +28,7 @@
 #include <string>
 #include <vector>
 
+#include "absl/log/check.h"
 #include "absl/strings/match.h"
 #include "absl/strings/str_cat.h"
 #include "absl/strings/str_format.h"
@@ -38,6 +37,7 @@
 #include <grpc/impl/channel_arg_names.h>
 #include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 #include <grpc/support/string_util.h>
 
 #include "src/core/lib/gpr/useful.h"
@@ -428,7 +428,7 @@ grpc_channel_args* grpc_channel_args_copy_and_add_and_remove(
   for (size_t i = 0; i < num_to_add; ++i) {
     dst->args[dst_idx++] = copy_arg(&to_add[i]);
   }
-  GPR_ASSERT(dst_idx == dst->num_args);
+  CHECK(dst_idx == dst->num_args);
   return dst;
 }
 
@@ -700,7 +700,7 @@ grpc_channel_args_client_channel_creation_mutator g_mutator = nullptr;
 
 void grpc_channel_args_set_client_channel_creation_mutator(
     grpc_channel_args_client_channel_creation_mutator cb) {
-  GPR_DEBUG_ASSERT(g_mutator == nullptr);
+  DCHECK_EQ(g_mutator, nullptr);
   g_mutator = cb;
 }
 grpc_channel_args_client_channel_creation_mutator

data/src/core/lib/channel/channel_args.h

@@ -19,8 +19,6 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_ARGS_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_ARGS_H
 
-#include <grpc/support/port_platform.h>
-
 #include <stddef.h>
 #include <stdint.h>
 
@@ -37,6 +35,7 @@
 
 #include <grpc/event_engine/event_engine.h>
 #include <grpc/grpc.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/avl/avl.h"
 #include "src/core/lib/gpr/useful.h"

data/src/core/lib/channel/channel_args_preconditioning.cc

@@ -12,12 +12,12 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_args_preconditioning.h"
 
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 void ChannelArgsPreconditioning::Builder::RegisterStage(Stage stage) {

data/src/core/lib/channel/channel_args_preconditioning.h

@@ -15,12 +15,11 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_ARGS_PRECONDITIONING_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_ARGS_PRECONDITIONING_H
 
-#include <grpc/support/port_platform.h>
-
 #include <functional>
 #include <vector>
 
 #include <grpc/grpc.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/channel/channel_args.h"
 

data/src/core/lib/channel/channel_stack.cc

@@ -16,8 +16,6 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_stack.h"
 
 #include <stdint.h>
@@ -25,7 +23,10 @@
 #include <memory>
 #include <utility>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/channel/channel_fwd.h"
@@ -68,8 +69,8 @@ size_t grpc_channel_stack_size(const grpc_channel_filter** filters,
                                                sizeof(grpc_channel_element));
   size_t i;
 
-  GPR_ASSERT((GPR_MAX_ALIGNMENT & (GPR_MAX_ALIGNMENT - 1)) == 0 &&
-             "GPR_MAX_ALIGNMENT must be a power of two");
+  CHECK((GPR_MAX_ALIGNMENT & (GPR_MAX_ALIGNMENT - 1)) == 0)
+      << "GPR_MAX_ALIGNMENT must be a power of two";
 
   // add the size for each filter
   for (i = 0; i < filter_count; i++) {
@@ -168,9 +169,9 @@ grpc_error_handle grpc_channel_stack_init(
     call_size += GPR_ROUND_UP_TO_ALIGNMENT_SIZE(filters[i]->sizeof_call_data);
   }
 
-  GPR_ASSERT(user_data > (char*)stack);
-  GPR_ASSERT((uintptr_t)(user_data - (char*)stack) ==
-             grpc_channel_stack_size(filters, filter_count));
+  CHECK(user_data > (char*)stack);
+  CHECK((uintptr_t)(user_data - (char*)stack) ==
+        grpc_channel_stack_size(filters, filter_count));
 
   stack->call_stack_size = call_size;
   return first_error;

data/src/core/lib/channel/channel_stack.h

@@ -44,8 +44,6 @@
 // it can have an effect on the call status.
 //
 
-#include <grpc/support/port_platform.h>
-
 #include <stddef.h>
 
 #include <functional>
@@ -56,6 +54,7 @@
 #include <grpc/slice.h>
 #include <grpc/status.h>
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 #include <grpc/support/time.h>
 
 #include "src/core/lib/channel/channel_args.h"

data/src/core/lib/channel/channel_stack_builder.cc

@@ -16,12 +16,12 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_stack_builder.h"
 
 #include <algorithm>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/channel/channel_args.h"
 
 namespace grpc_core {

data/src/core/lib/channel/channel_stack_builder.h

@@ -15,14 +15,14 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_STACK_BUILDER_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_STACK_BUILDER_H
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 #include <vector>
 
 #include "absl/status/statusor.h"
 #include "absl/strings/string_view.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/channel/channel_fwd.h"
 #include "src/core/lib/gprpp/ref_counted_ptr.h"

data/src/core/lib/channel/channel_stack_builder_impl.cc

@@ -16,8 +16,6 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_stack_builder_impl.h"
 
 #include <string.h>
@@ -36,6 +34,7 @@
 
 #include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/channel/channel_fwd.h"
 #include "src/core/lib/channel/channel_stack.h"
@@ -96,43 +95,37 @@ const grpc_channel_filter* PromiseTracingFilterFor(
               },
               /* init_call: */
               [](grpc_channel_element* elem, CallSpineInterface* call) {
+                auto* c = DownCast<PipeBasedCallSpine*>(call);
                 auto* source_filter =
                     static_cast<const DerivedFilter*>(elem->filter)->filter;
-                call->client_initial_metadata().receiver.InterceptAndMap(
+                c->client_initial_metadata().receiver.InterceptAndMap(
                     [source_filter](ClientMetadataHandle md) {
                       gpr_log(GPR_DEBUG, "%s[%s] OnClientInitialMetadata: %s",
                               GetContext<Activity>()->DebugTag().c_str(),
                               source_filter->name, md->DebugString().c_str());
                       return md;
                     });
-                call->client_to_server_messages().receiver.InterceptAndMap(
+                c->client_to_server_messages().receiver.InterceptAndMap(
                     [source_filter](MessageHandle msg) {
                       gpr_log(GPR_DEBUG, "%s[%s] OnClientToServerMessage: %s",
                               GetContext<Activity>()->DebugTag().c_str(),
                               source_filter->name, msg->DebugString().c_str());
                       return msg;
                     });
-                call->server_initial_metadata().sender.InterceptAndMap(
+                c->server_initial_metadata().sender.InterceptAndMap(
                     [source_filter](ServerMetadataHandle md) {
                       gpr_log(GPR_DEBUG, "%s[%s] OnServerInitialMetadata: %s",
                               GetContext<Activity>()->DebugTag().c_str(),
                               source_filter->name, md->DebugString().c_str());
                       return md;
                     });
-                call->server_to_client_messages().sender.InterceptAndMap(
+                c->server_to_client_messages().sender.InterceptAndMap(
                     [source_filter](MessageHandle msg) {
                       gpr_log(GPR_DEBUG, "%s[%s] OnServerToClientMessage: %s",
                               GetContext<Activity>()->DebugTag().c_str(),
                               source_filter->name, msg->DebugString().c_str());
                       return msg;
                     });
-                call->server_trailing_metadata().sender.InterceptAndMap(
-                    [source_filter](ServerMetadataHandle md) {
-                      gpr_log(GPR_DEBUG, "%s[%s] OnServerTrailingMetadata: %s",
-                              GetContext<Activity>()->DebugTag().c_str(),
-                              source_filter->name, md->DebugString().c_str());
-                      return md;
-                    });
               },
               grpc_channel_next_op,
               /* sizeof_call_data: */ 0,

data/src/core/lib/channel/channel_stack_builder_impl.h

@@ -15,10 +15,10 @@
 #ifndef GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_STACK_BUILDER_IMPL_H
 #define GRPC_SRC_CORE_LIB_CHANNEL_CHANNEL_STACK_BUILDER_IMPL_H
 
-#include <grpc/support/port_platform.h>
-
 #include "absl/status/statusor.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/channel/channel_fwd.h"
 #include "src/core/lib/channel/channel_stack_builder.h"
 #include "src/core/lib/gprpp/ref_counted_ptr.h"

data/src/core/lib/channel/channel_stack_trace.cc

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/channel_stack_trace.h"
 
+#include <grpc/support/port_platform.h>
+
 grpc_core::TraceFlag grpc_trace_channel_stack(false, "channel_stack");

data/src/core/lib/channel/connected_channel.cc

@@ -16,8 +16,6 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/connected_channel.h"
 
 #include <inttypes.h>
@@ -28,6 +26,7 @@
 #include <type_traits>
 #include <utility>
 
+#include "absl/log/check.h"
 #include "absl/status/status.h"
 #include "absl/status/statusor.h"
 #include "absl/types/optional.h"
@@ -36,6 +35,7 @@
 #include <grpc/status.h>
 #include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/channel/call_finalization.h"
 #include "src/core/lib/channel/channel_args.h"
@@ -225,7 +225,7 @@ static void connected_channel_destroy_call_elem(
 static grpc_error_handle connected_channel_init_channel_elem(
     grpc_channel_element* elem, grpc_channel_element_args* args) {
   channel_data* cd = static_cast<channel_data*>(elem->channel_data);
-  GPR_ASSERT(args->is_last);
+  CHECK(args->is_last);
   cd->transport = args->channel_args.GetObject<grpc_core::Transport>();
   return absl::OkStatus();
 }
@@ -464,8 +464,7 @@ ArenaPromise<ServerMetadataHandle> MakeClientCallPromise(Transport* transport,
       [](absl::Status) {});
   // Start a promise to receive server initial metadata and then forward it up
   // through the receiving pipe.
-  auto server_initial_metadata =
-      GetContext<Arena>()->MakePooled<ServerMetadata>();
+  auto server_initial_metadata = Arena::MakePooled<ServerMetadata>();
   party->Spawn(
       "recv_initial_metadata",
       TrySeq(GetContext<BatchBuilder>()->ReceiveServerInitialMetadata(
@@ -502,27 +501,25 @@ ArenaPromise<ServerMetadataHandle> MakeClientCallPromise(Transport* transport,
   // Create a promise that will receive server trailing metadata.
   // If this fails, we massage the error into metadata that we can report
   // upwards.
-  auto server_trailing_metadata =
-      GetContext<Arena>()->MakePooled<ServerMetadata>();
-  auto recv_trailing_metadata =
-      Map(GetContext<BatchBuilder>()->ReceiveServerTrailingMetadata(
-              stream->batch_target()),
-          [](absl::StatusOr<ServerMetadataHandle> status) mutable {
-            if (!status.ok()) {
-              auto server_trailing_metadata =
-                  GetContext<Arena>()->MakePooled<ServerMetadata>();
-              grpc_status_code status_code = GRPC_STATUS_UNKNOWN;
-              std::string message;
-              grpc_error_get_status(status.status(), Timestamp::InfFuture(),
-                                    &status_code, &message, nullptr, nullptr);
-              server_trailing_metadata->Set(GrpcStatusMetadata(), status_code);
-              server_trailing_metadata->Set(GrpcMessageMetadata(),
-                                            Slice::FromCopiedString(message));
-              return server_trailing_metadata;
-            } else {
-              return std::move(*status);
-            }
-          });
+  auto server_trailing_metadata = Arena::MakePooled<ServerMetadata>();
+  auto recv_trailing_metadata = Map(
+      GetContext<BatchBuilder>()->ReceiveServerTrailingMetadata(
+          stream->batch_target()),
+      [](absl::StatusOr<ServerMetadataHandle> status) mutable {
+        if (!status.ok()) {
+          auto server_trailing_metadata = Arena::MakePooled<ServerMetadata>();
+          grpc_status_code status_code = GRPC_STATUS_UNKNOWN;
+          std::string message;
+          grpc_error_get_status(status.status(), Timestamp::InfFuture(),
+                                &status_code, &message, nullptr, nullptr);
+          server_trailing_metadata->Set(GrpcStatusMetadata(), status_code);
+          server_trailing_metadata->Set(GrpcMessageMetadata(),
+                                        Slice::FromCopiedString(message));
+          return server_trailing_metadata;
+        } else {
+          return std::move(*status);
+        }
+      });
   // Finally the main call promise.
   // Concurrently: send initial metadata and receive messages, until BOTH
   // complete (or one fails).
@@ -785,8 +782,7 @@ ArenaPromise<ServerMetadataHandle> MakeServerCallPromise(
             if (status.ok()) {
               trailing_metadata = std::move(*status);
             } else {
-              trailing_metadata =
-                  GetContext<Arena>()->MakePooled<ClientMetadata>();
+              trailing_metadata = Arena::MakePooled<ClientMetadata>();
               grpc_status_code status_code = GRPC_STATUS_UNKNOWN;
               std::string message;
               grpc_error_get_status(status.status(), Timestamp::InfFuture(),
@@ -889,18 +885,7 @@ ArenaPromise<ServerMetadataHandle> MakeClientTransportCallPromise(
     Transport* transport, CallArgs call_args, NextPromiseFactory) {
   auto spine = GetContext<CallContext>()->MakeCallSpine(std::move(call_args));
   transport->client_transport()->StartCall(CallHandler{spine});
-  return Map(spine->server_trailing_metadata().receiver.Next(),
-             [](NextResult<ServerMetadataHandle> r) {
-               if (r.has_value()) {
-                 auto md = std::move(r.value());
-                 md->Set(GrpcStatusFromWire(), true);
-                 return md;
-               }
-               auto m = GetContext<Arena>()->MakePooled<ServerMetadata>();
-               m->Set(GrpcStatusMetadata(), GRPC_STATUS_CANCELLED);
-               m->Set(GrpcCallWasCancelled(), true);
-               return m;
-             });
+  return spine->PullServerTrailingMetadata();
 }
 
 const grpc_channel_filter kClientPromiseBasedTransportFilter =

data/src/core/lib/channel/context.h

@@ -29,9 +29,12 @@
 /// This enum represents the indexes into the array, where each index
 /// contains a different type of value.
 typedef enum {
+  /// grpc_call* associated with this context.
+  GRPC_CONTEXT_CALL = 0,
+
   /// Value is either a \a grpc_client_security_context or a
   /// \a grpc_server_security_context.
-  GRPC_CONTEXT_SECURITY = 0,
+  GRPC_CONTEXT_SECURITY,
 
   /// Value is a \a census_context.
   GRPC_CONTEXT_TRACING,
@@ -68,10 +71,35 @@ struct grpc_call_context_element {
 };
 
 namespace grpc_core {
+class Call;
+
 // Bind the legacy context array into the new style structure
 // TODO(ctiller): remove as we migrate these contexts to the new system.
 template <>
 struct ContextType<grpc_call_context_element> {};
+
+// Also as a transition step allow exposing a GetContext<T> that can peek into
+// the legacy context array.
+namespace promise_detail {
+template <typename T>
+struct OldStyleContext;
+
+template <>
+struct OldStyleContext<Call> {
+  static constexpr grpc_context_index kIndex = GRPC_CONTEXT_CALL;
+};
+
+template <typename T>
+class Context<T, absl::void_t<decltype(OldStyleContext<T>::kIndex)>> {
+ public:
+  static T* get() {
+    return static_cast<T*>(
+        GetContext<grpc_call_context_element>()[OldStyleContext<T>::kIndex]
+            .value);
+  }
+};
+
+}  // namespace promise_detail
 }  // namespace grpc_core
 
 #endif  // GRPC_SRC_CORE_LIB_CHANNEL_CONTEXT_H

data/src/core/lib/channel/metrics.cc

@@ -12,12 +12,14 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/channel/metrics.h"
 
 #include <memory>
 
+#include "absl/log/check.h"
+
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/crash.h"
 
 namespace grpc_core {
@@ -46,7 +48,7 @@ GlobalInstrumentsRegistry::RegisterUInt64Counter(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kUInt64;
   descriptor.instrument_type = InstrumentType::kCounter;
@@ -78,7 +80,7 @@ GlobalInstrumentsRegistry::RegisterDoubleCounter(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kDouble;
   descriptor.instrument_type = InstrumentType::kCounter;
@@ -110,7 +112,7 @@ GlobalInstrumentsRegistry::RegisterUInt64Histogram(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kUInt64;
   descriptor.instrument_type = InstrumentType::kHistogram;
@@ -142,7 +144,7 @@ GlobalInstrumentsRegistry::RegisterDoubleHistogram(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kDouble;
   descriptor.instrument_type = InstrumentType::kHistogram;
@@ -160,70 +162,6 @@ GlobalInstrumentsRegistry::RegisterDoubleHistogram(
   return handle;
 }
 
-GlobalInstrumentsRegistry::GlobalInt64GaugeHandle
-GlobalInstrumentsRegistry::RegisterInt64Gauge(
-    absl::string_view name, absl::string_view description,
-    absl::string_view unit, absl::Span<const absl::string_view> label_keys,
-    absl::Span<const absl::string_view> optional_label_keys,
-    bool enable_by_default) {
-  auto& instruments = GetInstrumentList();
-  for (const auto& descriptor : instruments) {
-    if (descriptor.name == name) {
-      Crash(
-          absl::StrFormat("Metric name %s has already been registered.", name));
-    }
-  }
-  uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
-  GlobalInstrumentDescriptor descriptor;
-  descriptor.value_type = ValueType::kInt64;
-  descriptor.instrument_type = InstrumentType::kGauge;
-  descriptor.index = index;
-  descriptor.enable_by_default = enable_by_default;
-  descriptor.name = name;
-  descriptor.description = description;
-  descriptor.unit = unit;
-  descriptor.label_keys = {label_keys.begin(), label_keys.end()};
-  descriptor.optional_label_keys = {optional_label_keys.begin(),
-                                    optional_label_keys.end()};
-  instruments.push_back(std::move(descriptor));
-  GlobalInt64GaugeHandle handle;
-  handle.index = index;
-  return handle;
-}
-
-GlobalInstrumentsRegistry::GlobalDoubleGaugeHandle
-GlobalInstrumentsRegistry::RegisterDoubleGauge(
-    absl::string_view name, absl::string_view description,
-    absl::string_view unit, absl::Span<const absl::string_view> label_keys,
-    absl::Span<const absl::string_view> optional_label_keys,
-    bool enable_by_default) {
-  auto& instruments = GetInstrumentList();
-  for (const auto& descriptor : instruments) {
-    if (descriptor.name == name) {
-      Crash(
-          absl::StrFormat("Metric name %s has already been registered.", name));
-    }
-  }
-  uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
-  GlobalInstrumentDescriptor descriptor;
-  descriptor.value_type = ValueType::kDouble;
-  descriptor.instrument_type = InstrumentType::kGauge;
-  descriptor.index = index;
-  descriptor.enable_by_default = enable_by_default;
-  descriptor.name = name;
-  descriptor.description = description;
-  descriptor.unit = unit;
-  descriptor.label_keys = {label_keys.begin(), label_keys.end()};
-  descriptor.optional_label_keys = {optional_label_keys.begin(),
-                                    optional_label_keys.end()};
-  instruments.push_back(std::move(descriptor));
-  GlobalDoubleGaugeHandle handle;
-  handle.index = index;
-  return handle;
-}
-
 GlobalInstrumentsRegistry::GlobalCallbackInt64GaugeHandle
 GlobalInstrumentsRegistry::RegisterCallbackInt64Gauge(
     absl::string_view name, absl::string_view description,
@@ -238,7 +176,7 @@ GlobalInstrumentsRegistry::RegisterCallbackInt64Gauge(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kInt64;
   descriptor.instrument_type = InstrumentType::kCallbackGauge;
@@ -270,7 +208,7 @@ GlobalInstrumentsRegistry::RegisterCallbackDoubleGauge(
     }
   }
   uint32_t index = instruments.size();
-  GPR_ASSERT(index < std::numeric_limits<uint32_t>::max());
+  CHECK_LT(index, std::numeric_limits<uint32_t>::max());
   GlobalInstrumentDescriptor descriptor;
   descriptor.value_type = ValueType::kDouble;
   descriptor.instrument_type = InstrumentType::kCallbackGauge;