[DNS Recovery]: Revert the cleanup timeout code.

uber-archive / node-statsd-client

Node.js client for statsd

ISC License

28 stars 9 forks source link

[DNS Recovery]: Revert the cleanup timeout code. #14

Closed Raynos closed 9 years ago

Raynos commented 9 years ago

The UDP socket timeout is being relied on in production. Removing it is an unexceptable breaking change. Removing it increases the number of file descriptors used in production.

We will have to live with this code forever instead of enforcing a statsd client must be strictly closed policy.

cc @sh1mmer @kriskowal

sh1mmer commented 9 years ago

other than logging. lgtm.

Raynos commented 9 years ago

cool.