Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Perf: limit degree of concurrency on ZipkinHttpReporter #27

Closed
Aaronontheweb opened this issue Apr 23, 2018 · 0 comments
Closed

Perf: limit degree of concurrency on ZipkinHttpReporter #27

Aaronontheweb opened this issue Apr 23, 2018 · 0 comments
Labels
enhancement New feature or request performance Performance-related design concerns

Comments

@Aaronontheweb
Copy link
Member

Kind of need #4 to do this properly. Some of our applications using Petabridge.Tracing.Zipkin are capable of toppling over a smaller Zipkin instance with little effort (although to be fair, the environment where this has occurred is Docker for Windows which isn't particularly robust.)

However, this will be an issue in sufficiently busy systems. I think we should consider using some type of exponential back-off-type mechanism or rate-limiting to help protect Zipkin endpoints from being overwhelmed by busy transports.

Honestly though, in a busy enough system HTTP ain't going to cut it. Might be better off just implementing gRPC #17 or Azure Event Hubs #18

@Aaronontheweb Aaronontheweb added enhancement New feature or request performance Performance-related design concerns labels Apr 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request performance Performance-related design concerns
Projects
None yet
Development

No branches or pull requests

1 participant