MarcGiffing / bucket4j-spring-boot-starter

Spring Boot Starter for Bucket4j
Apache License 2.0
298 stars 63 forks source link

Change log level when retrying connection with redis on JedisCacheListener #322

Closed wallysoncarvalho closed 1 week ago

wallysoncarvalho commented 1 month ago

Issue

I believe that an attempt at retrying a connection should not be considered as an error, instead we maybe should consider error if the retry attempt fails. What do you think we change the level of the log to WARN and log error only if the retry is interrupted?

Use case

On my team we consider log level ERROR a situation where we should act to fix, this is not one of these situations because the service "heals itself".

wallysoncarvalho commented 1 month ago

@MarcGiffing , can you take a look here?