Better async startup / shutdown hooks

LearnBoost / cluster

Node.JS multi-core server manager with plugins support.

http://learnboost.github.com/cluster

MIT License

2.29k stars 159 forks source link

Better async startup / shutdown hooks #20

Open tj opened 13 years ago

tj commented 13 years ago

currently the events fire, but they do not allow the shutdown to halt until cleanup is complete, removing files, sockets etc

jeffv commented 13 years ago

Was going to add this issue, but I see you've added it yourself. We've been running cluster in production for quite a while and those sockets can really pile up.

tj commented 13 years ago

it's a bit of a tough call, because if you defer the exit without some kind of timeout whatever task(s) may never finish, which is definitely far from ideal