Cleanup ServiceProxy#refreshServicePeerPartially for perf analysis

uber-archive / hyperbahn

Service discovery and routing for large scale microservice operations

MIT License

395 stars 57 forks source link

Closed jcorbin closed 8 years ago

jcorbin commented 8 years ago

In most recent flame graph under canary test:

this method took ~19% of process time
most of that 19% was due to an anonymous function that was the alternate to freshenPartialPeer

Hopefully this will provide us more clarity once / if we get down to tuning this path.

r @rf @Raynos

kriskowal commented 8 years ago

:ok:

Raynos commented 8 years ago

lgtm.