big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HADOOP-17116. Skip Retry INFO logging on first failover from a proxy

Browse Source
This commit is contained in:
Hanisha Koneru 2020-07-13 12:55:34 -07:00
parent 0427100b75
commit e62d8f8412
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/retry/RetryInvocationHandler.java
View File

@ -35,6 +35,7 @@
import java.lang.reflect.Proxy; import java.lang.reflect.Proxy;
import java.util.Arrays; import java.util.Arrays;
import java.util.Collections; import java.util.Collections;
import java.util.HashSet;
import java.util.Map; import java.util.Map;
/** /**
@ -312,6 +313,8 @@ public String toString() {
private volatile boolean hasSuccessfulCall = false; private volatile boolean hasSuccessfulCall = false;
private HashSet<String> failedAtLeastOnce = new HashSet<>();
private final RetryPolicy defaultPolicy; private final RetryPolicy defaultPolicy;
private final Map<String,RetryPolicy> methodNameToPolicyMap; private final Map<String,RetryPolicy> methodNameToPolicyMap;
@ -390,13 +393,19 @@ private RetryInfo handleException(final Method method, final int callId,
private void log(final Method method, final boolean isFailover, private void log(final Method method, final boolean isFailover,
final int failovers, final long delay, final Exception ex) { final int failovers, final long delay, final Exception ex) {
// log info if this has made some successful calls or boolean info = true;
// this is not the first failover // If this is the first failover to this proxy, skip logging at INFO level
final boolean info = hasSuccessfulCall || failovers != 0 if (!failedAtLeastOnce.contains(proxyDescriptor.getProxyInfo().toString()))
|| asyncCallHandler.hasSuccessfulCall(); {
failedAtLeastOnce.add(proxyDescriptor.getProxyInfo().toString());
// If successful calls were made to this proxy, log info even for first
// failover
info = hasSuccessfulCall || asyncCallHandler.hasSuccessfulCall();
if (!info && !LOG.isDebugEnabled()) { if (!info && !LOG.isDebugEnabled()) {
return; return;
} }
}
final StringBuilder b = new StringBuilder() final StringBuilder b = new StringBuilder()
.append(ex + ", while invoking ") .append(ex + ", while invoking ")