Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reply timeout handling is broken #101

Open
jappja opened this issue Jan 20, 2016 · 2 comments
Open

Reply timeout handling is broken #101

jappja opened this issue Jan 20, 2016 · 2 comments

Comments

@jappja
Copy link
Contributor

jappja commented Jan 20, 2016

TimeoutMixin's setTimeout is never called so timeouts are effectively disabled.
@jappja jappja mentioned this issue Jan 20, 2016
Closed
@arnimarj
Copy link

arnimarj commented Mar 4, 2016

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
Copy link
Owner

Cool! Looking forward for pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@arnimarj @IlyaSkriblovsky @jappja