Comment 3 for bug 777672

Revision history for this message
Hannu Valtonen (hannu-valtonen) wrote :

The testsuite ./testapp still passes. (i.e. 0 failed, 4 skipped, 1367 succeeded) I'm guessing the test set isn't really that comprehensive.

And yes I'm still using the old behavior system.

But anyway the two problems I see are that the next_retry time is never set and b) we'll never get to the switch statement at the end memcached_connect right now. i.e. we'll never reconnect. Basically it'll always end up in one of the "return MEMCACHED_SERVER_MARKED_DEAD; " code branches.

There are probably some issues with my patch too, it doesn't call update_continuum&friends in all the right places with this either.