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

Prevent starvation due to timeout requests #16

Merged
merged 1 commit into from Apr 28, 2014
Merged

Prevent starvation due to timeout requests #16

merged 1 commit into from Apr 28, 2014

Conversation

@kpdecker
Copy link
Contributor

kpdecker commented Apr 28, 2014

When requests timeout while they are still in the agent's request pool, they
depend on a variety of events to occur to cleanup the pool. When this does not
happen the sockets are left in a opened but unused state, counting against the
connection pool limit. When enough of these cases the limit will be exhausted
and no further requests can be processed.

When requests timeout while they are still in the agent's request pool, they depend on a variety of events to occur to cleanup the pool. When this does not happen the sockets are left in a opened but unused state, counting against the connection pool limit. When enough of these cases the limit will be exhausted and no further requests can be processed.
@hueniverse hueniverse added the bug label Apr 28, 2014
@hueniverse hueniverse added this to the 2.6.0 milestone Apr 28, 2014
@hueniverse hueniverse self-assigned this Apr 28, 2014
hueniverse added a commit that referenced this pull request Apr 28, 2014
Prevent starvation due to timeout requests
@hueniverse hueniverse merged commit d694571 into master Apr 28, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@kpdecker kpdecker deleted the agent-defer branch Apr 28, 2014
hueniverse added a commit that referenced this pull request Apr 28, 2014
@thebillkidy

This comment has been minimized.

Copy link

thebillkidy commented on 3f5d1e0 Apr 29, 2014

Thanks, this was a wanted commit :) I would have never figured this issue out on my own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.