grpc 1.76.0 → 1.78.0.pre1

data/src/core/lib/transport/transport.cc

@@ -26,9 +26,6 @@
 #include <memory>
 #include <new>
 
-#include "absl/status/status.h"
-#include "absl/strings/str_cat.h"
-#include "absl/strings/string_view.h"
 #include "src/core/lib/event_engine/default_event_engine.h"
 #include "src/core/lib/iomgr/exec_ctx.h"
 #include "src/core/lib/promise/for_each.h"
@@ -37,6 +34,9 @@
 #include "src/core/lib/slice/slice.h"
 #include "src/core/lib/transport/error_utils.h"
 #include "src/core/util/time.h"
+#include "absl/status/status.h"
+#include "absl/strings/str_cat.h"
+#include "absl/strings/string_view.h"
 
 void grpc_stream_destroy(grpc_stream_refcount* refcount) {
   if ((grpc_core::ExecCtx::Get()->flags() &

data/src/core/lib/transport/transport.h

@@ -33,16 +33,13 @@
 #include <string>
 #include <utility>
 
-#include "absl/functional/any_invocable.h"
-#include "absl/log/log.h"
-#include "absl/status/status.h"
-#include "absl/strings/string_view.h"
 #include "src/core/call/call_destination.h"
 #include "src/core/call/call_spine.h"
 #include "src/core/call/message.h"
 #include "src/core/call/metadata.h"
 #include "src/core/call/metadata_batch.h"
 #include "src/core/channelz/channelz.h"
+#include "src/core/ext/transport/chttp2/transport/http2_status.h"
 #include "src/core/lib/debug/trace.h"
 #include "src/core/lib/iomgr/call_combiner.h"
 #include "src/core/lib/iomgr/closure.h"
@@ -61,6 +58,10 @@
 #include "src/core/lib/transport/transport_fwd.h"
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted.h"
+#include "absl/functional/any_invocable.h"
+#include "absl/log/log.h"
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 
 // Minimum and maximum protocol accepted versions.
 #define GRPC_PROTOCOL_VERSION_MAX_MAJOR 2
@@ -507,6 +508,58 @@ class ServerTransport;
 
 class Transport : public InternallyRefCounted<Transport> {
  public:
+  // An interface used by channels or servers to watch the transport's state.
+  class StateWatcher : public RefCounted<StateWatcher> {
+   public:
+    ~StateWatcher() override = default;
+
+    // The list of reasons is defined in
+    // https://github.com/grpc/proposal/blob/master/A94-subchannel-otel-metrics.md.
+    // Note that we do not include the "subchannel shutdown" reason
+    // here, since that reason is not generated by the transport.
+    enum DisconnectReason {
+      kUnknown,
+      kGoaway,
+      kConnectionReset,
+      kConnectionTimedOut,
+      kConnectionAborted,
+      kSocketError,
+    };
+
+    struct DisconnectInfo {
+      DisconnectReason reason = kUnknown;
+      std::optional<http2::Http2ErrorCode> http2_error_code;
+      std::optional<Duration> keepalive_time;
+    };
+
+    // Called on disconnection or GOAWAY.  The channel or server must
+    // stop sending traffic to this transport.  The transport will
+    // automatically stop the watch after this.
+    virtual void OnDisconnect(absl::Status status,
+                              DisconnectInfo disconnect_info) = 0;
+
+    // A handle passed to the subchannel by the transport via
+    // OnPeerMaxConcurrentStreamsUpdate().  The subchannel must delete
+    // this handle when it has finished processing the update.
+    class MaxConcurrentStreamsUpdateDoneHandle {
+     public:
+      virtual ~MaxConcurrentStreamsUpdateDoneHandle() = default;
+    };
+
+    // Used on client transports only.
+    // Will be called once as soon as the watch is started to indicate
+    // the current value of the peer's MAX_CONCURRENT_STREAMS setting.
+    // Will then be called again whenever the peer changes this setting.
+    // The on_done handle must be deleted when the implementation is
+    // done processing the update.
+    virtual void OnPeerMaxConcurrentStreamsUpdate(
+        uint32_t max_concurrent_streams,
+        std::unique_ptr<MaxConcurrentStreamsUpdateDoneHandle> on_done) = 0;
+
+    // TODO(roth): Remove this as part of the EventEngine migration.
+    virtual grpc_pollset_set* interested_parties() const = 0;
+  };
+
   struct RawPointerChannelArgTag {};
   static absl::string_view ChannelArgName() { return GRPC_ARG_TRANSPORT; }
 
@@ -547,6 +600,8 @@ class Transport : public InternallyRefCounted<Transport> {
   // implementation of grpc_transport_perform_op
   virtual void PerformOp(grpc_transport_op* op) = 0;
 
+  // TODO(roth, ctiller): Migrate all callers to the new StartWatch()
+  // API and remove this.
   void StartConnectivityWatch(
       OrphanablePtr<ConnectivityStateWatcherInterface> watcher) {
     grpc_transport_op* op = grpc_make_transport_op(nullptr);
@@ -561,6 +616,9 @@ class Transport : public InternallyRefCounted<Transport> {
     PerformOp(op);
   }
 
+  virtual void StartWatch(RefCountedPtr<StateWatcher> watcher) = 0;
+  virtual void StopWatch(RefCountedPtr<StateWatcher> watcher) = 0;
+
   virtual RefCountedPtr<channelz::SocketNode> GetSocketNode() const = 0;
 };
 

data/src/core/lib/transport/transport_framing_endpoint_extension.h

@@ -17,9 +17,9 @@
 
 #include <grpc/support/port_platform.h>
 
+#include "src/core/lib/slice/slice_buffer.h"
 #include "absl/functional/any_invocable.h"
 #include "absl/strings/string_view.h"
-#include "src/core/lib/slice/slice_buffer.h"
 
 namespace grpc_core {
 

data/src/core/lib/transport/transport_op_string.cc

@@ -21,8 +21,6 @@
 #include <memory>
 #include <string>
 
-#include "absl/strings/str_cat.h"
-#include "absl/strings/str_format.h"
 #include "src/core/call/metadata_batch.h"
 #include "src/core/lib/channel/channel_fwd.h"
 #include "src/core/lib/slice/slice_buffer.h"
@@ -30,6 +28,8 @@
 #include "src/core/lib/transport/transport.h"
 #include "src/core/util/orphanable.h"
 #include "src/core/util/status_helper.h"
+#include "absl/strings/str_cat.h"
+#include "absl/strings/str_format.h"
 
 // These routines are here to facilitate debugging - they produce string
 // representations of various transport data structures

data/src/core/load_balancing/address_filtering.cc

@@ -21,10 +21,10 @@
 
 #include <utility>
 
-#include "absl/functional/function_ref.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/iomgr/resolved_address.h"
 #include "src/core/util/ref_counted_ptr.h"
+#include "absl/functional/function_ref.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/address_filtering.h

@@ -24,11 +24,11 @@
 #include <utility>
 #include <vector>
 
-#include "absl/status/statusor.h"
-#include "absl/strings/string_view.h"
 #include "src/core/resolver/endpoint_addresses.h"
 #include "src/core/util/ref_counted.h"
 #include "src/core/util/ref_counted_string.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/string_view.h"
 
 // The resolver returns a flat list of addresses.  When a hierarchy of
 // LB policies is in use, each leaf of the hierarchy will need a

data/src/core/load_balancing/backend_metric_parser.cc

@@ -21,11 +21,11 @@
 
 #include <map>
 
-#include "absl/strings/string_view.h"
 #include "upb/base/string_view.h"
 #include "upb/mem/arena.hpp"
 #include "upb/message/map.h"
 #include "xds/data/orca/v3/orca_load_report.upb.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/backend_metric_parser.h

@@ -20,8 +20,8 @@
 #include <grpc/support/port_platform.h>
 #include <stddef.h>
 
-#include "absl/strings/string_view.h"
 #include "src/core/load_balancing/backend_metric_data.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/child_policy_handler.cc

@@ -22,10 +22,6 @@
 #include <memory>
 #include <string>
 
-#include "absl/log/log.h"
-#include "absl/status/status.h"
-#include "absl/strings/str_cat.h"
-#include "absl/strings/string_view.h"
 #include "src/core/config/core_configuration.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/iomgr/pollset_set.h"
@@ -36,6 +32,10 @@
 #include "src/core/load_balancing/subchannel_interface.h"
 #include "src/core/util/debug_location.h"
 #include "src/core/util/grpc_check.h"
+#include "absl/log/log.h"
+#include "absl/status/status.h"
+#include "absl/strings/str_cat.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/child_policy_handler.h

@@ -20,13 +20,13 @@
 
 #include <utility>
 
-#include "absl/status/status.h"
-#include "absl/strings/string_view.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/debug/trace.h"
 #include "src/core/load_balancing/lb_policy.h"
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted_ptr.h"
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/delegating_helper.h

@@ -24,8 +24,6 @@
 
 #include <utility>
 
-#include "absl/status/status.h"
-#include "absl/strings/string_view.h"
 #include "src/core/credentials/transport/transport_credentials.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/iomgr/resolved_address.h"
@@ -33,6 +31,8 @@
 #include "src/core/load_balancing/subchannel_interface.h"
 #include "src/core/util/debug_location.h"
 #include "src/core/util/ref_counted_ptr.h"
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/endpoint_list.cc

@@ -26,10 +26,6 @@
 #include <utility>
 #include <vector>
 
-#include "absl/log/log.h"
-#include "absl/random/random.h"
-#include "absl/status/status.h"
-#include "absl/status/statusor.h"
 #include "src/core/config/core_configuration.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/experiments/experiments.h"
@@ -45,6 +41,10 @@
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted_ptr.h"
 #include "src/core/util/shared_bit_gen.h"
+#include "absl/log/log.h"
+#include "absl/random/random.h"
+#include "absl/status/status.h"
+#include "absl/status/statusor.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/endpoint_list.h

@@ -26,8 +26,6 @@
 #include <utility>
 #include <vector>
 
-#include "absl/functional/function_ref.h"
-#include "absl/status/status.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/iomgr/resolved_address.h"
 #include "src/core/load_balancing/lb_policy.h"
@@ -38,6 +36,8 @@
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted_ptr.h"
 #include "src/core/util/work_serializer.h"
+#include "absl/functional/function_ref.h"
+#include "absl/status/status.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/grpclb/client_load_reporting_filter.cc

@@ -52,12 +52,14 @@ void ClientLoadReportingFilter::Call::OnClientInitialMetadata(
     ClientMetadata& client_initial_metadata) {
   GRPC_LATENT_SEE_SCOPE(
       "ClientLoadReportingFilter::Call::OnClientInitialMetadata");
-  // Handle client initial metadata.
-  // Grab client stats object from metadata.
-  auto client_stats_md =
+  // Grab client stats object from metadata.  The metadata encodes only
+  // a raw pointer, but the LB policy will have returned a subchannel call
+  // tracker that is holding a ref to it, to ensure that it's alive here
+  // for us to take our own ref.
+  std::optional<GrpcLbClientStats*> client_stats_md =
       client_initial_metadata.Take(GrpcLbClientStatsMetadata());
-  if (client_stats_md.has_value()) {
-    client_stats_.reset(*client_stats_md);
+  if (client_stats_md.has_value() && *client_stats_md != nullptr) {
+    client_stats_ = (*client_stats_md)->Ref();
   }
 }
 

data/src/core/load_balancing/grpclb/client_load_reporting_filter.h

@@ -21,13 +21,13 @@
 
 #include <grpc/support/port_platform.h>
 
-#include "absl/status/statusor.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/channel/channel_fwd.h"
 #include "src/core/lib/channel/promise_based_filter.h"
 #include "src/core/lib/promise/arena_promise.h"
 #include "src/core/lib/transport/transport.h"
 #include "src/core/load_balancing/grpclb/grpclb_client_stats.h"
+#include "absl/status/statusor.h"
 
 namespace grpc_core {
 
@@ -47,6 +47,10 @@ class ClientLoadReportingFilter
     static inline const NoInterceptor OnClientToServerMessage;
     static inline const NoInterceptor OnClientToServerHalfClose;
     static inline const NoInterceptor OnFinalize;
+    channelz::PropertyList ChannelzProperties() {
+      return channelz::PropertyList().Set("saw_initial_metadata",
+                                          saw_initial_metadata_);
+    }
 
    private:
     RefCountedPtr<GrpcLbClientStats> client_stats_;

data/src/core/load_balancing/grpclb/grpclb.cc

@@ -80,16 +80,6 @@
 #include <variant>
 #include <vector>
 
-#include "absl/container/inlined_vector.h"
-#include "absl/functional/function_ref.h"
-#include "absl/log/globals.h"
-#include "absl/log/log.h"
-#include "absl/status/status.h"
-#include "absl/status/statusor.h"
-#include "absl/strings/str_cat.h"
-#include "absl/strings/str_format.h"
-#include "absl/strings/str_join.h"
-#include "absl/strings/string_view.h"
 #include "src/core/call/metadata_batch.h"
 #include "src/core/channelz/channelz.h"
 #include "src/core/client_channel/client_channel_filter.h"
@@ -141,6 +131,16 @@
 #include "src/core/util/validation_errors.h"
 #include "src/core/util/work_serializer.h"
 #include "upb/mem/arena.hpp"
+#include "absl/container/inlined_vector.h"
+#include "absl/functional/function_ref.h"
+#include "absl/log/globals.h"
+#include "absl/log/log.h"
+#include "absl/status/status.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/str_cat.h"
+#include "absl/strings/str_format.h"
+#include "absl/strings/str_join.h"
+#include "absl/strings/string_view.h"
 
 #define GRPC_GRPCLB_INITIAL_CONNECT_BACKOFF_SECONDS 1
 #define GRPC_GRPCLB_RECONNECT_BACKOFF_MULTIPLIER 1.6
@@ -417,10 +417,9 @@ class GrpcLb final : public LoadBalancingPolicy {
     PickResult Pick(PickArgs args) override;
 
    private:
-    // A subchannel call tracker that unrefs the GrpcLbClientStats object
-    // in the case where the subchannel call is never actually started,
-    // since the client load reporting filter will not be able to do it
-    // in that case.
+    // A subchannel call tracker that holds a ref to the
+    // GrpcLbClientStats object, to ensure that it still exists when the
+    // client load reporting filter sees it and takes its own ref to it.
     class SubchannelCallTracker final : public SubchannelCallTrackerInterface {
      public:
       SubchannelCallTracker(
@@ -429,16 +428,6 @@ class GrpcLb final : public LoadBalancingPolicy {
           : client_stats_(std::move(client_stats)),
             original_call_tracker_(std::move(original_call_tracker)) {}
 
-      void Start() override {
-        if (original_call_tracker_ != nullptr) {
-          original_call_tracker_->Start();
-        }
-        // If we're actually starting the subchannel call, then the
-        // client load reporting filter will take ownership of the ref
-        // passed down to it via metadata.
-        client_stats_.release();
-      }
-
       void Finish(FinishArgs args) override {
         if (original_call_tracker_ != nullptr) {
           original_call_tracker_->Finish(args);

data/src/core/load_balancing/grpclb/grpclb_client_stats.h

@@ -26,11 +26,11 @@
 #include <memory>
 #include <utility>
 
-#include "absl/base/thread_annotations.h"
-#include "absl/container/inlined_vector.h"
 #include "src/core/util/memory.h"
 #include "src/core/util/ref_counted.h"
 #include "src/core/util/sync.h"
+#include "absl/base/thread_annotations.h"
+#include "absl/container/inlined_vector.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/grpclb/load_balancer_api.cc

@@ -24,12 +24,12 @@
 
 #include <algorithm>
 
-#include "absl/log/log.h"
 #include "google/protobuf/duration.upb.h"
 #include "google/protobuf/timestamp.upb.h"
 #include "src/core/util/memory.h"
 #include "src/proto/grpc/lb/v1/load_balancer.upb.h"
 #include "upb/base/string_view.h"
+#include "absl/log/log.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/grpclb/load_balancer_api.h

@@ -24,10 +24,10 @@
 
 #include <vector>
 
-#include "absl/strings/string_view.h"
 #include "src/core/load_balancing/grpclb/grpclb_client_stats.h"
 #include "src/core/util/time.h"
 #include "upb/mem/arena.h"
+#include "absl/strings/string_view.h"
 
 #define GRPC_GRPCLB_SERVICE_NAME_MAX_LENGTH 128
 #define GRPC_GRPCLB_SERVER_IP_ADDRESS_MAX_SIZE 16

data/src/core/load_balancing/health_check_client.cc

@@ -30,11 +30,6 @@
 #include <type_traits>
 #include <utility>
 
-#include "absl/log/log.h"
-#include "absl/status/status.h"
-#include "absl/status/statusor.h"
-#include "absl/strings/str_cat.h"
-#include "absl/strings/string_view.h"
 #include "src/core/channelz/channel_trace.h"
 #include "src/core/client_channel/client_channel_internal.h"
 #include "src/core/client_channel/subchannel.h"
@@ -60,6 +55,11 @@
 #include "src/proto/grpc/health/v1/health.upb.h"
 #include "upb/base/string_view.h"
 #include "upb/mem/arena.hpp"
+#include "absl/log/log.h"
+#include "absl/status/status.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/str_cat.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 
@@ -283,6 +283,10 @@ class HealthProducer::ConnectivityWatcher final
     producer_->OnConnectivityStateChange(state, status);
   }
 
+  void OnKeepaliveUpdate(Duration) override {}
+
+  uint32_t max_connections_per_subchannel() const override { return 1; }
+
   grpc_pollset_set* interested_parties() override {
     return producer_->interested_parties_;
   }

data/src/core/load_balancing/health_check_client_internal.h

@@ -27,9 +27,6 @@
 #include <string>
 #include <utility>
 
-#include "absl/base/thread_annotations.h"
-#include "absl/status/status.h"
-#include "absl/strings/string_view.h"
 #include "src/core/client_channel/subchannel.h"
 #include "src/core/client_channel/subchannel_interface_internal.h"
 #include "src/core/client_channel/subchannel_stream_client.h"
@@ -41,6 +38,9 @@
 #include "src/core/util/sync.h"
 #include "src/core/util/unique_type_name.h"
 #include "src/core/util/work_serializer.h"
+#include "absl/base/thread_annotations.h"
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/lb_policy.h

@@ -31,11 +31,6 @@
 #include <utility>
 #include <variant>
 
-#include "absl/base/thread_annotations.h"
-#include "absl/container/inlined_vector.h"
-#include "absl/status/status.h"
-#include "absl/status/statusor.h"
-#include "absl/strings/string_view.h"
 #include "src/core/lib/channel/channel_args.h"
 #include "src/core/lib/debug/trace.h"
 #include "src/core/lib/iomgr/iomgr_fwd.h"
@@ -51,6 +46,11 @@
 #include "src/core/util/ref_counted_ptr.h"
 #include "src/core/util/sync.h"
 #include "src/core/util/work_serializer.h"
+#include "absl/base/thread_annotations.h"
+#include "absl/container/inlined_vector.h"
+#include "absl/status/status.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 
@@ -178,13 +178,16 @@ class LoadBalancingPolicy : public InternallyRefCounted<LoadBalancingPolicy> {
    public:
     virtual ~SubchannelCallTrackerInterface() = default;
 
-    /// Called when a subchannel call is started after an LB pick.
-    virtual void Start() = 0;
-
     /// Called when a subchannel call is completed.
     /// The metadata may be modified by the implementation.  However, the
     /// implementation does not take ownership, so any data that needs to be
     /// used after returning must be copied.
+    ///
+    /// Note that when the picker returns a complete pick, it's possible
+    /// that the returned subchannel has already lost its connection, in
+    /// which case the channel will queue the pick.  In that case,
+    /// the SubchannelCallTrackerInterface object will be destroyed
+    /// without ever calling Finish().
     struct FinishArgs {
       absl::string_view peer_address;
       absl::Status status;

data/src/core/load_balancing/lb_policy_factory.h

@@ -19,12 +19,12 @@
 
 #include <grpc/support/port_platform.h>
 
-#include "absl/status/statusor.h"
-#include "absl/strings/string_view.h"
 #include "src/core/load_balancing/lb_policy.h"
 #include "src/core/util/json/json.h"
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted_ptr.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/lb_policy_registry.cc

@@ -25,14 +25,14 @@
 #include <utility>
 #include <vector>
 
+#include "src/core/load_balancing/lb_policy.h"
+#include "src/core/util/grpc_check.h"
 #include "absl/log/log.h"
 #include "absl/status/status.h"
 #include "absl/strings/str_cat.h"
 #include "absl/strings/str_format.h"
 #include "absl/strings/str_join.h"
 #include "absl/strings/string_view.h"
-#include "src/core/load_balancing/lb_policy.h"
-#include "src/core/util/grpc_check.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/lb_policy_registry.h

@@ -22,13 +22,13 @@
 #include <map>
 #include <memory>
 
-#include "absl/status/statusor.h"
-#include "absl/strings/string_view.h"
 #include "src/core/load_balancing/lb_policy.h"
 #include "src/core/load_balancing/lb_policy_factory.h"
 #include "src/core/util/json/json.h"
 #include "src/core/util/orphanable.h"
 #include "src/core/util/ref_counted_ptr.h"
+#include "absl/status/statusor.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 

data/src/core/load_balancing/oob_backend_metric.cc

@@ -29,9 +29,6 @@
 #include <utility>
 #include <vector>
 
-#include "absl/log/log.h"
-#include "absl/status/status.h"
-#include "absl/strings/string_view.h"
 #include "google/protobuf/duration.upb.h"
 #include "src/core/channelz/channel_trace.h"
 #include "src/core/client_channel/subchannel.h"
@@ -54,6 +51,9 @@
 #include "src/core/util/time.h"
 #include "upb/mem/arena.hpp"
 #include "xds/service/orca/v3/orca.upb.h"
+#include "absl/log/log.h"
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 
 namespace grpc_core {
 
@@ -77,6 +77,10 @@ class OrcaProducer::ConnectivityWatcher final
     producer_->OnConnectivityStateChange(state);
   }
 
+  void OnKeepaliveUpdate(Duration) override {}
+
+  uint32_t max_connections_per_subchannel() const override { return 1; }
+
   grpc_pollset_set* interested_parties() override {
     return interested_parties_;
   }