Skip to content

Commit 63d5d9e

Browse files
authored
Merge pull request #435 from yidongnan/fix/issue-423
Improve NameResolverRegistry registration logs
2 parents 2bc07f8 + b984355 commit 63d5d9e

File tree

3 files changed

+11
-6
lines changed

3 files changed

+11
-6
lines changed

build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ buildscript {
99
}
1010
}
1111
ext {
12-
projectVersion = '2.10.1.RELEASE'
12+
projectVersion = '2.11.0-SNAPSHOT'
1313

1414
// https://github.com/grpc/grpc-java/releases
1515
grpcVersion = '1.31.1'

grpc-client-spring-boot-autoconfigure/src/main/java/net/devh/boot/grpc/client/nameresolver/NameResolverRegistration.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@ public void register(NameResolverRegistry registry) {
5858
for (NameResolverProvider provider : this.providers) {
5959
try {
6060
registry.register(provider);
61-
log.info("{} is available -> Added to the NameResolverRegistry", provider);
61+
log.debug("{} is available -> Added to the NameResolverRegistry", provider);
6262
} catch (IllegalArgumentException e) {
63-
log.info("{} is not available -> Not added to the NameResolverRegistry", provider);
63+
log.debug("{} is not available -> Not added to the NameResolverRegistry", provider);
6464
}
6565
}
6666
}
@@ -70,7 +70,7 @@ public void destroy() {
7070
for (NameResolverRegistry registry : this.registries) {
7171
for (NameResolverProvider provider : this.providers) {
7272
registry.deregister(provider);
73-
log.info("{} was removed from the NameResolverRegistry", provider);
73+
log.debug("{} was removed from the NameResolverRegistry", provider);
7474
}
7575
}
7676
this.registries.clear();

grpc-server-spring-boot-autoconfigure/src/main/java/net/devh/boot/grpc/server/nameresolver/SelfNameResolverFactory.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,12 @@ public class SelfNameResolverFactory extends NameResolverProvider {
4444
*
4545
* @param properties The properties used to resolve this server's address.
4646
*/
47-
public SelfNameResolverFactory(GrpcServerProperties properties) {
47+
public SelfNameResolverFactory(final GrpcServerProperties properties) {
4848
this.properties = properties;
4949
}
5050

5151
@Override
52-
public NameResolver newNameResolver(URI targetUri, Args args) {
52+
public NameResolver newNameResolver(final URI targetUri, final Args args) {
5353
if (SELF_SCHEME.equals(targetUri.getScheme()) || targetUri.toString().equals(SELF_SCHEME)) {
5454
return new SelfNameResolver(this.properties, args);
5555
}
@@ -71,4 +71,9 @@ protected int priority() {
7171
return 0; // Lowest priority
7272
}
7373

74+
@Override
75+
public String toString() {
76+
return "SelfNameResolverFactory [scheme=" + getDefaultScheme() + "]";
77+
}
78+
7479
}

0 commit comments

Comments
 (0)