Refactor Redis config to use redis.Options struct

Simplified Redis configuration by consolidating individual parameters (address, password, and DB) into the redis.Options struct. This improves code readability, reduces redundancy, and aligns with the Redis client API's recommended usage.

Signed-off-by: Tyler Lisowski <[email protected]>

Author: relyt0925

pkg/kv-cache/kvblock-indexer.go

@@ -59,16 +59,15 @@ var _ KVBlockIndexer = &RedisKVBlockIndexer{}
 
 // RedisKVBlockIndexerConfig holds the configuration for the RedisKVBlockIndexer.
 type RedisKVBlockIndexerConfig struct {
-	RedisAddr     string
-	RedisPassword string
-	RedisDB       int
+	RedisOpt *redis.Options
 }
 
 func defaultRedisKVBlockIndexerConfig() *RedisKVBlockIndexerConfig {
 	return &RedisKVBlockIndexerConfig{
-		RedisAddr:     "localhost:6379",
-		RedisPassword: "",
-		RedisDB:       0,
+		RedisOpt: &redis.Options{
+			Addr: "localhost:6379",
+			DB:   0,
+		},
 	}
 }
 
@@ -83,11 +82,7 @@ func NewRedisKVBlockIndexer(config *KVBlockIndexerConfig) (KVBlockIndexer, error
 		config = DefaultKVBlockIndexerConfig()
 	}
 
-	redisClient := redis.NewClient(&redis.Options{
-		Addr:     config.RedisAddr,
-		Password: config.RedisPassword,
-		DB:       config.RedisDB,
-	})
+	redisClient := redis.NewClient(config.RedisOpt)
 
 	_, err := redisClient.Ping(context.Background()).Result()
 	if err != nil {

tests/e2e/redis_mock/e2e_suite_test.go

@@ -59,7 +59,9 @@ func (s *KVCacheSuite) SetupTest() {
 	s.Require().NoError(err)
 
 	s.config = kvcache.NewDefaultConfig()
-	s.config.KVBlockIndexerConfig.RedisAddr = s.server.Addr()
+	s.config.KVBlockIndexerConfig.RedisOpt = &redis.Options{
+		Addr: s.server.Addr(),
+	}
 	s.config.PrefixStoreConfig.BlockSize = 4
 	s.config.TokenProcessorConfig.ChunkSize = 4