From 1e27a9e7fa76935ccdd045e6231c748ab6b5df71 Mon Sep 17 00:00:00 2001 From: Ananda Ray Date: Tue, 2 Jul 2024 12:21:59 +0530 Subject: [PATCH] Fix confusing log message on breach of overallTimeout duration --- .../src/main/java/io/helidon/faulttolerance/RetryImpl.java | 7 ++++--- 1 file changed, 4 insertions(+), 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 index a6e8c41dfa4..d8d8100030a 100644 --- 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 @@ -95,9 +95,10 @@ public T invoke(Supplier 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());