Reply timeout handling is broken
jappja opened this issue · 2 comments
jappja commented
TimeoutMixin's setTimeout is never called so timeouts are effectively disabled.
arnimarj commented
Hi. I've been hit hard by this problem, and I'm working on a solution at https://github.com/arnimarj/txredisapi/commits/master
I've put it through a pathological case where I've put the replyTimeout to 10ms, and it seems to deal with it just fine. Once I've added unit-tests I'll submit a pull request.
IlyaSkriblovsky commented
Cool! Looking forward for pull request