Fix confusing log message on breach of overallTimeout duration (#8936)

helidon-io · Jul 16, 2024 · d0b2d63 · d0b2d63
1 parent ba5635a
commit d0b2d63
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/fault-tolerance/fault-tolerance/src/main/java/io/helidon/faulttolerance/RetryImpl.java b/fault-tolerance/fault-tolerance/src/main/java/io/helidon/faulttolerance/RetryImpl.java
@@ -100,9 +100,10 @@ public <T> T invoke(Supplier<? extends T> supplier) {
 
     public void checkTimeout(RetryContext<?> context, long nanoTime) {
         if ((nanoTime - context.startedNanos) > maxTimeNanos) {
-            RetryTimeoutException te = new RetryTimeoutException("Execution took too long. Already executing: "
-                                                                         + TimeUnit.NANOSECONDS.toMillis(nanoTime)
-                                                                         + " ms, must timeout after: "
+            RetryTimeoutException te = new RetryTimeoutException("Execution took too long. Already executing for: "
+                                                                         + TimeUnit.NANOSECONDS.toMillis(
+                                                                             nanoTime - context.startedNanos)
+                                                                         + " ms, must be lower than overallTimeout duration of: "
                                                                          + TimeUnit.NANOSECONDS.toMillis(maxTimeNanos)
                                                                          + " ms.",
                                                                  context.throwable());