coopernurse / node-pool

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

Improve "'Resource not currently part of this pool'" errors #206

Open sandfox opened 7 years ago

sandfox commented 7 years ago

As noted in #203 destroy and release both can reject with `'Resource not currently part of this pool' Errors.

The error is actually a bit of lie as the internal logic is actually just checking if the resource is currently loaned out.

We should add further error messages to cover all the failure modes

this is a semver major change