Skip to content

Commit d227520

Browse files
zhangkun83creamsoup
authored andcommitted
xds: fix lint warnings in test code (#6559)
1 parent 90969b3 commit d227520

File tree

1 file changed

+18
-14
lines changed

1 file changed

+18
-14
lines changed

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

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -374,12 +374,12 @@ public void resolve_resourceNewlyAdded() {
374374
}
375375

376376
/**
377-
* Builds an LDS DiscoveryResponse containing the mapping of given host name (with port if any)
378-
* to the given cluster name directly in-line. Clients receiving this response is able to
379-
* resolve cluster name for the given hostname:port immediately.
377+
* Builds an LDS DiscoveryResponse containing the mapping of given host name (with port if any) to
378+
* the given cluster name directly in-line. Clients receiving this response is able to resolve
379+
* cluster name for the given hostname:port immediately.
380380
*/
381-
private DiscoveryResponse buildLdsResponseForCluster(String versionInfo,
382-
String hostName, int port, String clusterName, String nonce) {
381+
private static DiscoveryResponse buildLdsResponseForCluster(
382+
String versionInfo, String hostName, int port, String clusterName, String nonce) {
383383
String ldsResourceName = port == -1 ? hostName : hostName + ":" + port;
384384
List<Any> listeners = ImmutableList.of(
385385
Any.pack(buildListener(ldsResourceName,
@@ -396,12 +396,12 @@ private DiscoveryResponse buildLdsResponseForCluster(String versionInfo,
396396
}
397397

398398
/**
399-
* Builds an LDS DiscoveryResponse containing the mapping of given host name (with port if any)
400-
* to the given RDS resource name. Clients receiving this response is able to send an RDS
401-
* request for resolving the cluster name for the given hostname:port.
399+
* Builds an LDS DiscoveryResponse containing the mapping of given host name (with port if any) to
400+
* the given RDS resource name. Clients receiving this response is able to send an RDS request for
401+
* resolving the cluster name for the given hostname:port.
402402
*/
403-
private DiscoveryResponse buildLdsResponseForRdsResource(String versionInfo,
404-
String hostName, int port, String routeConfigName, String nonce) {
403+
private static DiscoveryResponse buildLdsResponseForRdsResource(
404+
String versionInfo, String hostName, int port, String routeConfigName, String nonce) {
405405
String ldsResourceName = port == -1 ? hostName : hostName + ":" + port;
406406
Rds rdsConfig =
407407
Rds.newBuilder()
@@ -418,11 +418,15 @@ private DiscoveryResponse buildLdsResponseForRdsResource(String versionInfo,
418418
}
419419

420420
/**
421-
* Builds an RDS DiscoveryResponse containing the mapping of given route config name to the
422-
* given cluster name under.
421+
* Builds an RDS DiscoveryResponse containing the mapping of given route config name to the given
422+
* cluster name under.
423423
*/
424-
private DiscoveryResponse buildRdsResponseForCluster(String versionInfo,
425-
String routeConfigName, String hostName, String clusterName, String nonce) {
424+
private static DiscoveryResponse buildRdsResponseForCluster(
425+
String versionInfo,
426+
String routeConfigName,
427+
String hostName,
428+
String clusterName,
429+
String nonce) {
426430
List<Any> routeConfigs = ImmutableList.of(
427431
Any.pack(
428432
buildRouteConfiguration(

0 commit comments

Comments
 (0)