causiq / logary

Logs and metrics are one! Professional logging, metrics and analytics for your apps.
https://docs.logary.tech/
Other
528 stars 71 forks source link

use span-based api when exporting metric infos #404

Open lust4life opened 5 years ago

lust4life commented 5 years ago

I'm not going to argue here at all, but we should consider using netcoreapp2.2 in the future, and defaulting to the non-allocating Span-based API:s.

_Originally posted by @haf in https://github.com/_render_node/MDE3OlB1bGxSZXF1ZXN0UmV2aWV3MjE0NjgzNzAx/pull_request_reviews/more_threads_

lust4life commented 5 years ago

seems right now, the span-based api in StringBuilder doesn't give more good here. since what we do is format data to string then append to stringbuilder, then generate bytes to suave's response content.

maybe these would help :

After these have been done and expose, I think we can make it easier to get the benefit. What do you think ?

haf commented 5 years ago

I think that if we use the Span based API, I can replace the server from underneath to one that supports writing straight to the socket in the future.

I don't think we'll get much benefit immediately though.