fix: pass the configurer

Author: AgraVator

api/src/main/java/io/grpc/ChildChannelConfigurer.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2025 The gRPC Authors
+ * Copyright 2026 The gRPC Authors
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -60,8 +60,8 @@ public interface ChildChannelConfigurer extends Consumer<ManagedChannelBuilder<?
    * before {@link ManagedChannelBuilder#build()} is called.
    *
    * <p>Note: The provided {@code builder} is generic (`?`). Implementations should use
-   * universal configuration methods (like {@code intercept()}, {@code userAgent()}) rather
-   * than casting to specific implementation types.
+   * universal configuration methods (like {@code intercept()}, {@code userAgent()}) on the
+   * builder rather than casting it to specific implementation types.
    *
    * @param builder the mutable channel builder for the new child channel
    */

api/src/main/java/io/grpc/ChildChannelConfigurers.java

@@ -242,17 +242,6 @@ public T disableServiceConfigLookUp() {
     return thisT();
   }
 
-  @Override
-  public T configureChannel(ManagedChannel parentChannel) {
-    delegate().configureChannel(parentChannel);
-    return thisT();
-  }
-
-  @Override
-  public T configureChannel(Server parentServer) {
-    delegate().configureChannel(parentServer);
-    return thisT();
-  }
 
   @Override
   public T childChannelConfigurer(ChildChannelConfigurer childChannelConfigurer) {

api/src/main/java/io/grpc/ForwardingChannelBuilder.java

@@ -269,17 +269,6 @@ public <X> T setNameResolverArg(NameResolver.Args.Key<X> key, X value) {
     return thisT();
   }
 
-  @Override
-  public T configureChannel(ManagedChannel parentChannel) {
-    delegate().configureChannel(parentChannel);
-    return thisT();
-  }
-
-  @Override
-  public T configureChannel(Server parentServer) {
-    delegate().configureChannel(parentServer);
-    return thisT();
-  }
 
   @Override
   public T childChannelConfigurer(ChildChannelConfigurer childChannelConfigurer) {

api/src/main/java/io/grpc/ForwardingChannelBuilder2.java

@@ -192,12 +192,6 @@ public T setBinaryLog(BinaryLog binaryLog) {
     return thisT();
   }
 
-  @Override
-  public T configureChannel(Server parentServer) {
-    delegate().configureChannel(parentServer);
-    return thisT();
-  }
-
   @Override
   public T childChannelConfigurer(ChildChannelConfigurer childChannelConfigurer) {
     delegate().childChannelConfigurer(childChannelConfigurer);

api/src/main/java/io/grpc/ForwardingServerBuilder.java

@@ -85,23 +85,6 @@ public ConnectivityState getState(boolean requestConnection) {
     throw new UnsupportedOperationException("Not implemented");
   }
 
-  /**
-   * Returns the configurer for child channels.
-   *
-   * <p>This method is intended for use by the internal gRPC infrastructure (specifically
-   * load balancers and the channel builder) to propagate configuration to child channels.
-   * Application code should not call this method.
-   *
-   * @return the configurer, or {@code noOp()} if none is set.
-   * @since 1.79.0
-   */
-  @Internal
-  public ChildChannelConfigurer getChildChannelConfigurer() {
-    // Return noOP() by default so we don't break existing custom ManagedChannel implementations
-    // (like wrappers or mocks) that don't override this method.
-    return ChildChannelConfigurers.noOp();
-  }
-
   /**
    * Registers a one-off callback that will be run if the connectivity state of the channel diverges
    * from the given {@code source}, which is typically what has just been returned by {@link

api/src/main/java/io/grpc/ManagedChannel.java

@@ -657,37 +657,6 @@ public <X> T setNameResolverArg(NameResolver.Args.Key<X> key, X value) {
     throw new UnsupportedOperationException();
   }
 
-  /**
-   * Configures this builder using settings derived from an existing parent channel.
-   *
-   * <p>This method is typically used by internal components (like LoadBalancers) when creating
-   * child channels to ensure they inherit relevant configuration (like the
-   * {@link ChildChannelConfigurer}) from the parent.
-   *
-   * @param parentChannel the channel to inherit configuration from
-   * @return this
-   * @since 1.79.0
-   */
-  @ExperimentalApi("https://github.com/grpc/grpc-java/issues/12574")
-  public T configureChannel(ManagedChannel parentChannel) {
-    throw new UnsupportedOperationException();
-  }
-
-  /**
-   * Configures this builder using settings derived from an existing parent server.
-   *
-   * <p>This method is typically used by internal components (like LoadBalancers) when creating
-   * child channels to ensure they inherit relevant configuration (like the
-   * {@link ChildChannelConfigurer}) from the parent.
-   *
-   * @param parentServer the server to inherit configuration from
-   * @return this
-   * @since 1.79.0
-   */
-  @ExperimentalApi("https://github.com/grpc/grpc-java/issues/12574")
-  public T configureChannel(Server parentServer) {
-    throw new UnsupportedOperationException();
-  }
 
   /**
    * Sets a configurer that will be applied to all internal child channels created by this channel.

api/src/main/java/io/grpc/ManagedChannelBuilder.java

@@ -323,7 +323,7 @@ public static final class Args {
     @Nullable private final MetricRecorder metricRecorder;
     @Nullable private final NameResolverRegistry nameResolverRegistry;
     @Nullable private final IdentityHashMap<Key<?>, Object> customArgs;
-    @Nullable private final ManagedChannel parentChannel;
+    @Nullable private final ChildChannelConfigurer childChannelConfigurer;
 
     private Args(Builder builder) {
       this.defaultPort = checkNotNull(builder.defaultPort, "defaultPort not set");
@@ -338,7 +338,7 @@ private Args(Builder builder) {
       this.metricRecorder = builder.metricRecorder;
       this.nameResolverRegistry = builder.nameResolverRegistry;
       this.customArgs = cloneCustomArgs(builder.customArgs);
-      this.parentChannel = builder.parentChannel;
+      this.childChannelConfigurer = builder.childChannelConfigurer;
     }
 
     /**
@@ -438,11 +438,14 @@ public ChannelLogger getChannelLogger() {
     }
 
     /**
-     * Returns the parent {@link ManagedChannel} served by this NameResolver.
+     * Returns the configurer for child channels.
+     *
+     * @since 1.81.0
      */
+    @Nullable
     @Internal
-    public ManagedChannel getParentChannel() {
-      return parentChannel;
+    public ChildChannelConfigurer getChildChannelConfigurer() {
+      return childChannelConfigurer;
     }
 
     /**
@@ -554,7 +557,7 @@ public static final class Builder {
       private MetricRecorder metricRecorder;
       private NameResolverRegistry nameResolverRegistry;
       private IdentityHashMap<Key<?>, Object> customArgs;
-      private ManagedChannel parentChannel;
+      private ChildChannelConfigurer childChannelConfigurer;
 
       Builder() {
       }
@@ -671,12 +674,12 @@ public Builder setNameResolverRegistry(NameResolverRegistry registry) {
       }
 
       /**
-       * See {@link Args#parentChannel}. This is an optional field.
+       * See {@link Args#getChildChannelConfigurer()}. This is an optional field.
        *
-       * @since 1.79.0
+       * @since 1.81.0
        */
-      public Builder setParentChannel(ManagedChannel parentChannel) {
-        this.parentChannel = parentChannel;
+      public Builder setChildChannelConfigurer(ChildChannelConfigurer childChannelConfigurer) {
+        this.childChannelConfigurer = childChannelConfigurer;
         return this;
       }
 

api/src/main/java/io/grpc/NameResolver.java

@@ -179,20 +179,4 @@ public List<ServerServiceDefinition> getMutableServices() {
    */
   public abstract void awaitTermination() throws InterruptedException;
 
-  /**
-   * Returns the configurer for child channels.
-   *
-   * <p>This method is intended for use by the internal gRPC infrastructure
-   * to propagate configuration to child channels.
-   * Application code should not call this method.
-   *
-   * @return the configurer, or {@code noOp()} if none is set.
-   * @since 1.79.0
-   */
-  @Internal
-  public ChildChannelConfigurer getChildChannelConfigurer() {
-    // Return noOp() by default so we don't break existing custom ManagedChannel implementations
-    // (like wrappers or mocks) that don't override this method.
-    return ChildChannelConfigurers.noOp();
-  }
 }

api/src/main/java/io/grpc/Server.java

@@ -424,21 +424,6 @@ public T setBinaryLog(BinaryLog binaryLog) {
     throw new UnsupportedOperationException();
   }
 
-  /**
-   * Configures this builder using settings derived from an existing parent server.
-   *
-   * <p>This method is typically used by internal components when creating
-   * child channels to ensure they inherit relevant configuration (like the
-   * {@link ChildChannelConfigurer}) from the parent.
-   *
-   * @param parentServer the server to inherit configuration from
-   * @return this
-   * @since 1.79.0
-   */
-  @ExperimentalApi("https://github.com/grpc/grpc-java/issues/12574")
-  public T configureChannel(Server parentServer) {
-    throw new UnsupportedOperationException();
-  }
 
   /**
    * Sets a configurer that will be applied to all internal child channels created by this server.