Skip to content

Remove Stale connections that have not been used for idletimeout. #1749

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

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

Remove Stale connections that have not been used for idletimeout. #1749

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
e46ef6e
Remove Stale connections that have not been used for idletimeout peri…
aankur Jun 6, 2017
105cee0
fix Lint errors
aankur Jun 7, 2017
8896669
fix space space-infix-ops lint error
aankur Jun 7, 2017
dd404fe
Fix eslint errors
aankur Aug 30, 2017
b6afaf5
Merge branch 'master' into feature-idletimeout-pool
aankur Aug 30, 2017
ead7dd4
fix lint errors
aankur Aug 30, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix lint errors
  • Loading branch information
@aankur
aankur committed Aug 30, 2017
commit ead7dd4298c5f3ea9fd8f43b6b8646de8fe1e024
2 changes: 1 addition & 1 deletion lib/Pool.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ function recycleStaleConnections() {
this._freeConnections[i].destroy();
i--;
} else {
if(min == null) {
if (min == null) {
min = now - lastused;
} else {
if ((now - lastused) < min) {
Expand Down