Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run peer refresh and discovery #33

Closed
cheme opened this issue Mar 20, 2015 · 1 comment
Closed

Run peer refresh and discovery #33

cheme opened this issue Mar 20, 2015 · 1 comment
Labels
Milestone

Comments

@cheme
Copy link
Owner

cheme commented Mar 20, 2015

Currently peer are only added, when received.
it is ok as long as we periodically publish peer, this is another issue and should be manage by application or see propagate keycache [https://github.com//issues/35].
This is ok but when route reply with not enough result we need new now so
TODO

  • add peer discovery and connection refresh on get_closest_peer with not enough result -> route need new interface
    • one to send disconnect peer which should be refresh (get_closest should return a pair with connected vec and (non connected vec * factor) to complete in not enough peer) -> the mult factor should be in a rule and added to get_closest (at start it should be omit).
    • one to tell if peer discovery is needed (and the number needed) : otherwhise as long as we send to n peer we could have an route with only n peer which for some network could be insecure. This should be called periodically (param by rule), by peermanagement process (see querycache cleaning).
@cheme cheme added the route label Mar 20, 2015
@cheme cheme added this to the alpha milestone Mar 20, 2015
@cheme
Copy link
Owner Author

cheme commented Nov 22, 2015

Connection is retry once in client before failure.
Peer refresh is possible with peermanager (see new route interface to get multiple peer then just ping).

@cheme cheme closed this as completed Nov 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant