search

coopernurse / node-pool

Generic resource pooling for node.js
2.37k stars 259 forks source link

fix(Pool): continue iterating when reaching DoubleLinkedList end #243

Closed DiegoRBaquero closed 5 years ago

DiegoRBaquero commented 5 years ago

After reaching the linked list end and reseting it, we should continue iterating to evict idle resources.

sandfox commented 5 years ago

LGMT - That's definitely what it should be doing.

DiegoRBaquero commented 5 years ago

Hey @sandfox , thanks for the quick handling. Already using in production (through sequelize v4).

It's now evicting 3 idle connections instead of 1, as supposed, and when they reach their actual idle time, instead of always!

Thanks again! Lmk if you ever need any help with any issues.