search

LearnBoost / up

Zero-downtime reloads and requests load balancer based on distribute.
540 stars 73 forks source link

Add support for asynchronous worker startup #29

Closed mugami-ast closed 12 years ago

mugami-ast commented 12 years ago

This adds a new option called assumeReady (default: true) that controls whether up should automatically add the new worker to the pool, or if it is up to the worker to asynchronous tell up when it is ready for to handle new requests.

Signed-off-by: Asbjørn Sloth Tønnesen ast@veridu.com

rauchg commented 12 years ago

Pretty hot. Needs a test for ultimate hawtness

mugami-ast commented 12 years ago

I omitted the test because the existing tests were broken. I have rebased this branch onto pull request #30, so only the last 2 commits are unique for this pull request.

mugami-ast commented 12 years ago

Rebased on to new 0.2.0 master, and still parsing all tests. Whats the hold up on merging this?

mugami-ast commented 12 years ago

Rebased onto new 0.2.1 master

jamessharp commented 11 years ago

Hi guys - is there any reason why this feature isn't in the npm package of up?