Ran Shidlansik
Ran Shidlansik
@ThachNgocTran - I cannot seem to be able to reproduce this issue on both unstable and the latest 6.2 (6.2.7 [e6f6709](https://github.com/redis/redis/commit/e6f67092f8d4d81761a60c46011d1ff1dc3c2628)) - can you maybe check on which OSS commit...
@oranagra/@madolson this is following the issue https://github.com/redis/redis/issues/7551 lots of small changes in this PR, but I think it brings some order onto things. @oranagra in case the change of bpop...
Just to report that a week ago I run this change on the daily CI: https://github.com/ranshid/redis/actions/runs/2656180654 some errors but I am still not sure it is related to this change
@oranagra , @madolson , @enjoy-binbin I forced push in order to rebase my version on top of the updated repo (merge would have made some salad of commits) I was...
> @ranshid i don't understand why a merge commit would be a problem. i'd prefer you avoid that next time. anyway, water under the bridge now, so please tell me...
I have read the new comments and will address all of them. However due to some work related obligations and the holidays I will only be able to work on...
> @ranshid i've reviewed some of this PR but in order to continue please undo some code moving (db.c, blocked.c, remove blocked.h) so the diff won't be as big @guybe7...
> @oranagra One easy (but maybe not very memory efficient) way to do that without any additional functions is to use `{BaseKey}DerivedKey` to name derived keys off base keys. I...
@liyaoxuan I am also tracking this PR as it seems to provide functionality we are all missing - thank you for doing it! Wanted to ask regarding you current implementation:...
@liyaoxuan I can see that you changed the producer/consumer design and went with a "per-client" qps control approach. This if fine, but I would like to ask again regarding the...