Open DrLynch opened 1 month ago
Check on default pool size/behavior and whether changing pool affects connections.
From past work it seems this is happening by default. What we don't know yet is whether this makes a major difference in operations. That should be tested with experimentation.
The Redis client we use, redis-py, internally initializes a connection pool with 2^31 maximum connections by default (see source code for reference here).
Based upon recent system tests it appears that we are spending a lot of time on redis connection requests under the async calls. Some of that has to do with the ping and may need to be moved around but it is also worth checking whether the redis-asyncio system is also using pools and is able to handle pool connections. At present we do not explicitly use or manage the connection pooling nor do we turn on more than one connection. We need to check if that is doable and then to check what changes can be made.