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.

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

fix not applying :timeout in query options #239

Merged
merged 1 commit into from
Oct 9, 2018

Conversation

rixmann
Copy link

@rixmann rixmann commented Sep 27, 2018

The :timeout option parameter in DBConnection callbacks was not forwarded to
the message receiving function.
Instead the configured general :timeout was used.

Added two testcases which will block for ~1.5 seconds.

closes #162

The :timeout option parameter in DBConnection callbacks was not forwarded to
the message receiving function.
Instead the configured general :timeout was used.

Added two testcases which will block for ~1.5 seconds.

closes xerions#162
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.275% when pulling f48e99b on rixmann:fix_timeout_handling_for_requests into eac3104 on xerions:master.

@0xAX 0xAX merged commit 0527548 into xerions:master Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

per query timeouts not respected
3 participants