search

uber-archive / hyperbahn

Service discovery and routing for large scale microservice operations
MIT License
396 stars 57 forks source link

Add staleness checking for partial affinity workers #261

Closed jcorbin closed 8 years ago

jcorbin commented 8 years ago

Rather than always create peers by side-effect, check the partialRange.workers against actual serviceChannel peeers, remove stale ones, and log loudly about it.

jcorbin commented 8 years ago

Dear reviewers,

I'm going to split out isomorphic work first (#262), let me know if you want the "don't create" semantic change separated from the "staleness check" change.

Raynos commented 8 years ago

Looks fine.

Raynos commented 8 years ago

It looks ok.