search

uber-archive / hyperbahn

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

remove per-worker timings #289

Closed blampe closed 8 years ago

blampe commented 8 years ago

Per-worker latency timings represent a majority of our metrics but rolling these up to the host level is sufficient.

@kriskowal @sloppyfocus

kriskowal commented 8 years ago

@Raynos please sign. CI will pass soon.

Raynos commented 8 years ago

This is wrong location for this fix. Look in batch-stats instead.

https://github.com/uber/hyperbahn/blob/master/clients/batch-stats.js

kriskowal commented 8 years ago

@Raynos i.e., axe this line? https://github.com/uber/hyperbahn/blob/master/clients/batch-stats.js#L67

blampe commented 8 years ago

@kriskowal I'll let you take it from here :)

Raynos commented 8 years ago

lgtm.