Emit disconnect on worker disconnect.

doxout / recluster

Node clustering library with support for zero downtime reloading

522 stars 44 forks source link

Emit disconnect on worker disconnect. #6

Closed sandinmyjoints closed 11 years ago

spion commented 11 years ago

Looks good, I'll be merging it.

Just a friendly warning: the event emitter API is not public for a reason - its error prone right now. Specifically calling removeAllListeners on some events (or without an argument) may leave the cluster in a broken state where nothing works because the cluster uses its own events to implement its internal mechanics.

I do plan on fixing that.

sandinmyjoints commented 11 years ago

OK, thanks for the tip!