search

helidon-io / helidon

Java libraries for writing microservices
https://helidon.io
Apache License 2.0
3.5k stars 566 forks source link

Fix confusing log message on breach of overallTimeout duration #8936

Closed anandbibek closed 2 months ago

anandbibek commented 3 months ago

Description

The log message printed out on breach of overallTimeout property in Helidon fault-tolerance Retry module is confusing and misleading.

Existing

io.helidon.faulttolerance.RetryTimeoutException: Execution took too long. Already executing: 80344961 ms, must timeout after: 1000 ms.

Updated

io.helidon.faulttolerance.RetryTimeoutException: Execution took too long. Already executing for: 1460 ms, must be lower than overallTimeout duration of: 1000 ms.

Documentation

No doc impact

anandbibek commented 3 months ago

Hi @spericas, could you please take a look at this?