RestForwarder: Remove redundant logRetry method
The log.debug statement can simply be inlined. There is no need
to call log.isDebugEnabled since concatenation of the arguments
is delegated to the log.debug method, which already includes a
check for debug level being enabled.
Change-Id: I2a0fe5cf7ed27100afab74d1eed72f112a57cdff
diff --git a/src/main/java/com/ericsson/gerrit/plugins/highavailability/forwarder/rest/RestForwarder.java b/src/main/java/com/ericsson/gerrit/plugins/highavailability/forwarder/rest/RestForwarder.java
index c937e66..c38e849 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/highavailability/forwarder/rest/RestForwarder.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/highavailability/forwarder/rest/RestForwarder.java
@@ -140,7 +140,7 @@
return false;
}
- logRetry(e);
+ log.debug("Retrying to {} caused by '{}'", name, e);
try {
Thread.sleep(cfg.http().retryInterval());
} catch (InterruptedException ie) {
@@ -168,11 +168,5 @@
boolean isRecoverable(IOException e) {
return !(e instanceof SSLException);
}
-
- void logRetry(Throwable cause) {
- if (log.isDebugEnabled()) {
- log.debug("Retrying to {} caused by '{}'", name, cause);
- }
- }
}
}
