search

tobymao / saq

Simple Async Queues
https://saq-py.readthedocs.io/en/latest/
MIT License
577 stars 39 forks source link

Require redis >=4.2,<4.6 #64

Closed lqmanh closed 1 year ago

lqmanh commented 1 year ago

Resolve #63

tobymao commented 1 year ago

this is not correct because redis broke cancel catching in 4.4 look at

https://github.com/redis/redis-py/pull/2104

tobymao commented 1 year ago

this is still open https://github.com/redis/redis-py/issues/2511

lqmanh commented 1 year ago

Okay this seems really bad. Guess I have to find some workaround then :((( Thanks for your quick response 👍🏼

kartzke commented 1 year ago

Looks like this PR may fix the issue: https://github.com/redis/redis-py/pull/2695

It's been hanging for 3 months. There's been some dialog in the last few days, but it needs the attention of a redis-py maintainer.