Add `Client_main.find_master'`

Incubaid / arakoon

A consistent distributed key-value store

http://arakoon.org

Apache License 2.0

84 stars 17 forks source link

Add `Client_main.find_master'` #479

Closed NicolasT closed 10 years ago

NicolasT commented 10 years ago

This patch adds Client_main.find_master', which should be a more resilient master lookup algorithm.

It also reworks Client_main.find_master to use the new find_master', but keep the behaviour of the original action regarding error handling.

NicolasT commented 10 years ago

[x] CI: arakoon-git-launcher 356
[x] Attempt to connect to all IP addresses of a node concurrently?

domsj commented 10 years ago

I think we should definitely try connecting on all ips of a node, and while it could be done concurrently doing it sequentially would already be an important improvement too (and actually be good enough for me).

Anyway, this definitely looks like an improvement so far.

NicolasT commented 10 years ago

Implemented concurrent connecting.

NicolasT commented 10 years ago

[ ] arakoon-git-launcher 361

domsj commented 10 years ago

LG if CI is ok