diff options
| author | Kun Zhang <zhangkun83@users.noreply.github.com> | 2018-09-07 09:44:10 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-09-07 09:44:10 -0700 |
| commit | e1c6cadb50312f0dc810541b1e245b9adca89709 (patch) | |
| tree | 608697df824eff8517040729355612e0e65df036 /grpclb | |
| parent | 189991012bbff42b975c51045c32efceaa07f462 (diff) | |
| download | grpc-grpc-java-e1c6cadb50312f0dc810541b1e245b9adca89709.tar.gz | |
grpclb: more useful debug logs. (#4831)
The addresses from the string dump of the LoadBalanceResponse proto is
in binary format and not human-readable. We will log the
BackendAddressGroups when using a new list from the balancer. The
original logging of LoadBalanceResponse is downgraded to FINER level.
Diffstat (limited to 'grpclb')
| -rw-r--r-- | grpclb/src/main/java/io/grpc/grpclb/BackendAddressGroup.java | 5 | ||||
| -rw-r--r-- | grpclb/src/main/java/io/grpc/grpclb/GrpclbState.java | 4 |
2 files changed, 8 insertions, 1 deletions
diff --git a/grpclb/src/main/java/io/grpc/grpclb/BackendAddressGroup.java b/grpclb/src/main/java/io/grpc/grpclb/BackendAddressGroup.java index 1e38d611e..7398be44a 100644 --- a/grpclb/src/main/java/io/grpc/grpclb/BackendAddressGroup.java +++ b/grpclb/src/main/java/io/grpc/grpclb/BackendAddressGroup.java @@ -39,4 +39,9 @@ final class BackendAddressGroup { String getToken() { return token; } + + @Override + public String toString() { + return "[addrs=" + addresses + " token=" + token + "]"; + } } diff --git a/grpclb/src/main/java/io/grpc/grpclb/GrpclbState.java b/grpclb/src/main/java/io/grpc/grpclb/GrpclbState.java index 3ed723f55..bf170cd9a 100644 --- a/grpclb/src/main/java/io/grpc/grpclb/GrpclbState.java +++ b/grpclb/src/main/java/io/grpc/grpclb/GrpclbState.java @@ -344,6 +344,8 @@ final class GrpclbState { private void useRoundRobinLists( List<DropEntry> newDropList, List<BackendAddressGroup> newBackendAddrList, @Nullable GrpclbClientLoadRecorder loadRecorder) { + logger.log(Level.FINE, "[{0}] Using round-robin list: {1}, droplist={2}", + new Object[] {logId, newBackendAddrList, newDropList}); HashMap<EquivalentAddressGroup, Subchannel> newSubchannelMap = new HashMap<EquivalentAddressGroup, Subchannel>(); List<BackendEntry> newBackendList = new ArrayList<BackendEntry>(); @@ -540,7 +542,7 @@ final class GrpclbState { if (closed) { return; } - logger.log(Level.FINE, "[{0}] Got an LB response: {1}", new Object[] {logId, response}); + logger.log(Level.FINER, "[{0}] Got an LB response: {1}", new Object[] {logId, response}); LoadBalanceResponseTypeCase typeCase = response.getLoadBalanceResponseTypeCase(); if (!initialResponseReceived) { |