Commands don't implement `CommandsProtocol.connection_pool`

[Avasam]

Version: redis-py 4.5.4

Platform: Any

Description:

The CommandsProtocol protocol at redis/typing.py has a connection_pool member. However, all commands that subclasses the protocol seem to be missing connection_pool in their implementation. This causes issues with type-checking tools expecting the commands to respect the Protocol.

[Avasam]

I could make a PR to fix it, but the problem is that I don't know what was the intention. Is the protocol wrong for wanting connection_pool ? Or are the commands wrong for not implementing it? Is the variable set dynamically later down the line? If so, where, and should connection_pool be potentially None instead of potentially undefined?

[github-actions]

This issue is marked stale. It will be closed in 30 days if it is not updated.

[Avasam]

Update. I can make a PR for this. I just need an answer first.

[petyaslavova]

Closing this issue as a duplicate of #3574 (I leave the newer one, because it is easier to find it). I'm planning to work on a fix soon.