Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

operation timeout #352

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

wandenberg
Copy link
Contributor

This is the same pull request as #329, just rebased (I'll close the old one)

Add a timeout to read socket to be possible stop the process when the server is taking too long to answer due to high load or a heavy query

@swistaczek
Copy link

👍 @wandenberg but Travis builds fail

@arthurnn
Copy link
Contributor

@wandenberg thanks for the PR, do you mind squashing the commits in one, and add a changelog entry?

@durran @estolfo can you guys have a look, does this make sense?

@wandenberg
Copy link
Contributor Author

@arthurnn I squashed in three commit because it were originally three different features.
If you want to I can squash in only one, but this way is better to understand what was done

@fbernier
Copy link

fbernier commented May 5, 2015

We use this in production and it does work as expected.

@wandenberg wandenberg force-pushed the operation_timeout branch from 7885008 to 1deb6e7 Compare June 3, 2015 13:48
@wandenberg wandenberg force-pushed the operation_timeout branch from 1deb6e7 to 470224c Compare June 3, 2015 14:16
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.59%) to 93.3% when pulling 1deb6e7 on wandenberg:operation_timeout into 68923e0 on mongoid:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 93.83% when pulling 1deb6e7 on wandenberg:operation_timeout into 68923e0 on mongoid:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 93.83% when pulling 470224c on wandenberg:operation_timeout into 68923e0 on mongoid:master.

@agis
Copy link

agis commented Sep 18, 2015

Any updates on this one? /cc @durran @estolfo

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants