From a87690d817546a39a66a1bcb5955513bb063e52f Mon Sep 17 00:00:00 2001 From: Katherine Date: Mon, 7 Jul 2025 12:12:44 -0400 Subject: [PATCH] Include Redis cluster and shard address in circuit breaker log --- .../textsecuregcm/redis/LettuceShardCircuitBreaker.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/redis/LettuceShardCircuitBreaker.java b/service/src/main/java/org/whispersystems/textsecuregcm/redis/LettuceShardCircuitBreaker.java index 65c3e70ab..ec6e307fd 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/redis/LettuceShardCircuitBreaker.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/redis/LettuceShardCircuitBreaker.java @@ -248,7 +248,7 @@ public class LettuceShardCircuitBreaker implements NettyCustomizer { // RedisNoScriptException doesn’t indicate a fault the breaker can protect if (throwable != null && !(throwable instanceof RedisNoScriptException)) { breaker.onError(durationNanos, TimeUnit.NANOSECONDS, throwable); - logger.warn("Command completed with error", throwable); + logger.warn("Command completed with error for: {}/{}", clusterName, shardAddress, throwable); } else { breaker.onSuccess(durationNanos, TimeUnit.NANOSECONDS); }