Use redis.asyncio instead of thread (!128) · Merge requests · Remi Rampin / taguette

Remi Rampin requested to merge redis-async into master Oct 08, 2022

Uses redis.asyncio instead of the client based on a background thread.

It turns out that the background thread was polling pretty heavily, which was causing the slowdown making the tests much slower than non-Redis and causing the CPU-intensive tests (e.g. hashing) to fail.

In addition, close the race-condition where events could be published between the time we check the database and the time we subscribe to Redis.

Use redis.asyncio instead of thread

Merge request reports