Ayush Sharma
Ayush Sharma
Hi, @madolson can you please take a look?
Addressed reviews and added UT to test that permissions are set correctly. Not sure which group to use in tests since using the default root group or creating new group...
Hi, I can pick this up. Can this be assigned to me?
>Move all client’s fields accessed by the IO threads to a sub-struct. Picking this up. cc: @uriyage since you mentioned this previously [here](https://github.com/valkey-io/valkey/pull/1405#discussion_r1885607905).
UTs pending atm.
Hey @uriyage, just curious , can you please explain the safety concerns a bit more i.e. why we would want to drain all jobs and inactivate all threads instead of...
Hey @uriyage @madolson , anything else needed to move this PR forward?
Before this feature, the only time we called `shutdownIoThread()` was on server shutdown. Considering this wasn't noticed until now, I don't think its too severe and needs a backport but...