Skip to content

Commit 56e4ce4

Browse files
authored
Rename AbstractXdsClient to ControlPlaneClient (#9934)
AbstractXdsClient is not abstract.
1 parent b55ecd5 commit 56e4ce4

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

xds/src/main/java/io/grpc/xds/AbstractXdsClient.java renamed to xds/src/main/java/io/grpc/xds/ControlPlaneClient.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
* Common base type for XdsClient implementations, which encapsulates the layer abstraction of
6161
* the xDS RPC stream.
6262
*/
63-
final class AbstractXdsClient {
63+
final class ControlPlaneClient {
6464

6565
public static final String CLOSED_BY_SERVER = "Closed by server";
6666
private final SynchronizationContext syncContext;
@@ -92,7 +92,7 @@ final class AbstractXdsClient {
9292

9393
/** An entity that manages ADS RPCs over a single channel. */
9494
// TODO: rename to XdsChannel
95-
AbstractXdsClient(
95+
ControlPlaneClient(
9696
XdsChannelFactory xdsChannelFactory,
9797
ServerInfo serverInfo,
9898
Node bootstrapNode,
@@ -388,7 +388,7 @@ void start() {
388388

389389
@Override
390390
public void beforeStart(ClientCallStreamObserver<DiscoveryRequest> requestStream) {
391-
requestStream.setOnReadyHandler(AbstractXdsClient.this::readyHandler);
391+
requestStream.setOnReadyHandler(ControlPlaneClient.this::readyHandler);
392392
}
393393

394394
@Override

xds/src/main/java/io/grpc/xds/XdsClientImpl.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public void uncaughtException(Thread t, Throwable e) {
9393
private final FilterRegistry filterRegistry = FilterRegistry.getDefaultRegistry();
9494
private final LoadBalancerRegistry loadBalancerRegistry
9595
= LoadBalancerRegistry.getDefaultRegistry();
96-
private final Map<ServerInfo, AbstractXdsClient> serverChannelMap = new HashMap<>();
96+
private final Map<ServerInfo, ControlPlaneClient> serverChannelMap = new HashMap<>();
9797
private final Map<XdsResourceType<? extends ResourceUpdate>,
9898
Map<String, ResourceSubscriber<? extends ResourceUpdate>>>
9999
resourceSubscribers = new HashMap<>();
@@ -144,7 +144,7 @@ private void maybeCreateXdsChannelWithLrs(ServerInfo serverInfo) {
144144
if (serverChannelMap.containsKey(serverInfo)) {
145145
return;
146146
}
147-
AbstractXdsClient xdsChannel = new AbstractXdsClient(
147+
ControlPlaneClient xdsChannel = new ControlPlaneClient(
148148
xdsChannelFactory,
149149
serverInfo,
150150
bootstrapInfo.node(),
@@ -218,7 +218,7 @@ public void run() {
218218
return;
219219
}
220220
isShutdown = true;
221-
for (AbstractXdsClient xdsChannel : serverChannelMap.values()) {
221+
for (ControlPlaneClient xdsChannel : serverChannelMap.values()) {
222222
xdsChannel.shutdown();
223223
}
224224
if (reportingLoad) {
@@ -490,7 +490,7 @@ private <T extends ResourceUpdate> void handleResourceUpdate(XdsResourceType.Arg
490490
*/
491491
private final class ResourceSubscriber<T extends ResourceUpdate> {
492492
@Nullable private final ServerInfo serverInfo;
493-
@Nullable private final AbstractXdsClient xdsChannel;
493+
@Nullable private final ControlPlaneClient xdsChannel;
494494
private final XdsResourceType<T> type;
495495
private final String resource;
496496
private final Set<ResourceWatcher<T>> watchers = new HashSet<>();
@@ -518,7 +518,7 @@ private final class ResourceSubscriber<T extends ResourceUpdate> {
518518
// is created but not yet requested because the client is in backoff.
519519
this.metadata = ResourceMetadata.newResourceMetadataUnknown();
520520

521-
AbstractXdsClient xdsChannelTemp = null;
521+
ControlPlaneClient xdsChannelTemp = null;
522522
try {
523523
maybeCreateXdsChannelWithLrs(serverInfo);
524524
xdsChannelTemp = serverChannelMap.get(serverInfo);

xds/src/test/java/io/grpc/xds/XdsClientImplTestBase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ public abstract class XdsClientImplTestBase {
158158
new FakeClock.TaskFilter() {
159159
@Override
160160
public boolean shouldAccept(Runnable command) {
161-
return command.toString().contains(AbstractXdsClient.RpcRetryTask.class.getSimpleName());
161+
return command.toString().contains(ControlPlaneClient.RpcRetryTask.class.getSimpleName());
162162
}
163163
};
164164

0 commit comments

Comments
 (0)