fix(redis): Implement redisNode interface for redis.Redis

[DengY11]

This change implements the missing Pipeline , TxPipelined , and Command methods on the redis.Redis type. This ensures that *redis.Redis satisfies the redisNode interface.

close #5175

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull Request Overview

This PR implements the missing Pipeline, TxPipelined, and Command methods on the redis.Redis type to ensure it fully satisfies the redisNode interface, addressing issue #5175.

Key changes:

• Added three new methods (Pipeline, TxPipelined, Command) to the Redis struct
• Added a new type alias CommandsInfoCmd for the redis command info result
• Updated and expanded test coverage for the new pipeline and command functionality

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
core/stores/redis/redis.go	Implements the three missing interface methods and adds CommandsInfoCmd type alias
core/stores/redis/redis_test.go	Replaces old TxPipeline test with comprehensive tests for all three new methods including error cases

[kevwan]

Is there any scenario that we need to run command in services?

[DengY11]

Is there any scenario that we need to run command in services?

Maybe we don't have a specific use case for this method, it was added to fully implement the required interface.

[kevwan]

What if we remove this method?

[DengY11]

What if we remove this method?

then Redis doesn't fully implement the RedisNode interface, methods like Blpop , BlpopCtx , and BlpopEx cannot accept Redis as parameters.