Reducing verbosity during BWS errors (#4311)

Stacktrace can be retrieved by increasing the log level

Signed-off-by: Jiri Peinlich <jiri.peinlich@gmail.com>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>

ethereum/eth/src/main/java/org/hyperledger/besu/ethereum/eth/sync/backwardsync/BackwardSyncContext.java

@@ -202,24 +202,24 @@ public class BackwardSyncContext {
             backwardSyncException -> {
               if (backwardSyncException.shouldRestart()) {
                 LOG.info(
-                    "Backward sync failed ({}). Current Peers: {}. Retrying in "
+                    "Backward sync failed ({}). Current Peers: {}. Retrying in {} milliseconds...",
-                        + millisBetweenRetries
+                    throwable.getMessage(),
-                        + " milliseconds...",
+                    ethContext.getEthPeers().peerCount(),
-                    backwardSyncException.getMessage(),
+                    millisBetweenRetries);
-                    ethContext.getEthPeers().peerCount());
-                return;
               } else {
                 debugLambda(
                     LOG, "Not recoverable backward sync exception {}", throwable::getMessage);
                 throw backwardSyncException;
               }
             },
-            () ->
+            () -> {
-                LOG.warn(
+              LOG.warn(
-                    "There was an uncaught exception during Backwards Sync. Retrying in "
+                  "Backward sync failed ({}). Current Peers: {}. Retrying in {} milliseconds...",
-                        + millisBetweenRetries
+                  throwable.getMessage(),
-                        + " milliseconds...",
+                  ethContext.getEthPeers().peerCount(),
-                    throwable));
+                  millisBetweenRetries);
+              LOG.debug("Exception details:", throwable);
+            });
   }
 
   private Optional<BackwardSyncException> extractBackwardSyncException(final Throwable throwable) {