From 63f0b0da68a72a2157bcc112a4ff5f2b9ea75731 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexander=20Aleksandrovi=C4=8D=20Klimov?= Date: Wed, 13 Aug 2025 17:14:16 +0200 Subject: [PATCH] redis.NewClientFromConfig(): remove obsolete workaround The underlying Redis client library doesn't blindly use eventually broken connections from the pool anymore. Hence, we don't have to re-try commands more often than the pool size. --- redis/client.go | 1 - 1 file changed, 1 deletion(-) diff --git a/redis/client.go b/redis/client.go index f13bd45e..13f2c2ed 100644 --- a/redis/client.go +++ b/redis/client.go @@ -76,7 +76,6 @@ func NewClientFromConfig(c *Config, logger *logging.Logger) (*Client, error) { client := redis.NewClient(options) options = client.Options() options.PoolSize = max(32, options.PoolSize) - options.MaxRetries = options.PoolSize + 1 // https://github.com/go-redis/redis/issues/1737 return NewClient(redis.NewClient(options), logger, &c.Options), nil }