vseloved / cl-redis

Redis client for Common Lisp
Other
188 stars 38 forks source link

[Feature] Configurable client-side timeouts #38

Open gmakris-rp opened 1 year ago

gmakris-rp commented 1 year ago

Hello!

I am opening this issue to add a few new features that would allow for greater control over the timeouts of both opening connections and individual operations.

As it is right now opening connections inherit the default timeout value of usocket:socket-connect. We could have more flexibility if that value was configurable. Additionally, a global timeout for all commands could be added, as well as the option to make the timeouts signal an error. I noticed some commands already support it as an argument but a default value for the rest would be really useful.

I have done a thorough review of the changes that need to be made in order to support this. They are very few and sensible defaults could be set so as to not affect existing users.