diff --git a/xds/src/main/java/io/grpc/xds/CdsLoadBalancer.java b/xds/src/main/java/io/grpc/xds/CdsLoadBalancer.java index fe5c2ebcec..4ad4776ccf 100644 --- a/xds/src/main/java/io/grpc/xds/CdsLoadBalancer.java +++ b/xds/src/main/java/io/grpc/xds/CdsLoadBalancer.java @@ -83,7 +83,7 @@ public final class CdsLoadBalancer extends LoadBalancer { @Override public void handleResolvedAddresses(ResolvedAddresses resolvedAddresses) { - channelLogger.log(ChannelLogLevel.DEBUG, "Received ResolvedAddresses '{0}'", resolvedAddresses); + channelLogger.log(ChannelLogLevel.DEBUG, "Received ResolvedAddresses {0}", resolvedAddresses); Attributes attributes = resolvedAddresses.getAttributes(); if (xdsClientRef == null) { xdsClientRef = resolvedAddresses.getAttributes().get(XdsAttributes.XDS_CLIENT_REF); @@ -133,7 +133,7 @@ public final class CdsLoadBalancer extends LoadBalancer { @Override public void handleNameResolutionError(Status error) { - channelLogger.log(ChannelLogLevel.ERROR, "Name resolution error: '{0}'", error); + channelLogger.log(ChannelLogLevel.ERROR, "Name resolution error: {0}", error); // Go into TRANSIENT_FAILURE if we have not yet received any cluster resource. Otherwise, // we keep running with the data we had previously. if (clusterWatcher == null) { @@ -302,7 +302,7 @@ public final class CdsLoadBalancer extends LoadBalancer { @Override public void onClusterChanged(ClusterUpdate newUpdate) { channelLogger.log( - ChannelLogLevel.DEBUG, "CDS load balancer received a cluster update: '{0}'", newUpdate); + ChannelLogLevel.DEBUG, "CDS load balancer received a cluster update: {0}", newUpdate); checkArgument( newUpdate.getLbPolicy().equals("round_robin"), "The load balancing policy in ClusterUpdate '%s' is not supported", newUpdate); @@ -329,8 +329,7 @@ public final class CdsLoadBalancer extends LoadBalancer { @Override public void onError(Status error) { - channelLogger.log( - ChannelLogLevel.ERROR, "CDS load balancer received an error: '{0}'", error); + channelLogger.log(ChannelLogLevel.ERROR, "CDS load balancer received an error: {0}", error); // Go into TRANSIENT_FAILURE if we have not yet created the child // policy (i.e., we have not yet received valid data for the cluster). Otherwise, diff --git a/xds/src/main/java/io/grpc/xds/LookasideLb.java b/xds/src/main/java/io/grpc/xds/LookasideLb.java index 68be0724e7..8bc3750e70 100644 --- a/xds/src/main/java/io/grpc/xds/LookasideLb.java +++ b/xds/src/main/java/io/grpc/xds/LookasideLb.java @@ -116,7 +116,7 @@ final class LookasideLb extends LoadBalancer { @Override public void handleResolvedAddresses(ResolvedAddresses resolvedAddresses) { - channelLogger.log(ChannelLogLevel.DEBUG, "Received ResolvedAddresses '{0}'", resolvedAddresses); + channelLogger.log(ChannelLogLevel.DEBUG, "Received ResolvedAddresses {0}", resolvedAddresses); Attributes attributes = resolvedAddresses.getAttributes(); XdsConfig newXdsConfig; @@ -237,7 +237,7 @@ final class LookasideLb extends LoadBalancer { @Override public void handleNameResolutionError(Status error) { - channelLogger.log(ChannelLogLevel.ERROR, "Name resolution error: '{0}'", error); + channelLogger.log(ChannelLogLevel.ERROR, "Name resolution error: {0}", error); // Go into TRANSIENT_FAILURE if we have not yet received any endpoint update. Otherwise, // we keep running with the data we had previously. if (endpointWatcher == null) { @@ -465,7 +465,7 @@ final class LookasideLb extends LoadBalancer { public void onEndpointChanged(EndpointUpdate endpointUpdate) { channelLogger.log( ChannelLogLevel.DEBUG, - "EDS load balancer received an endpoint update: '{0}'", + "EDS load balancer received an endpoint update: {0}", endpointUpdate); if (!firstEndpointUpdateReceived) { @@ -501,8 +501,7 @@ final class LookasideLb extends LoadBalancer { @Override public void onError(Status error) { - channelLogger.log( - ChannelLogLevel.ERROR, "EDS load balancer received an error: '{0}'", error); + channelLogger.log(ChannelLogLevel.ERROR, "EDS load balancer received an error: {0}", error); endpointUpdateCallback.onError(); } }