nico piderman

Thanks for starting this discussion! I tend to rely more on system level metrics like load and latency for making decisions around scaling, but I do think that exposing these...

We also have the pool timeout as well, so there could be a total of 4 timeouts :) It seems like a lot but I suppose most users would just...

Hi @learc83! Thanks for looking in to this, I would love to see an OpenTelemetryFinch library. My initial concern was that adding this `Registry.meta/2` call for every request would have...

Hi @GregMefford! If I am understanding correctly, you are referring to a case where someone wants to use multiple request_transformer functions? Since this function accepts a Request and returns a...

Maybe we should also formally specify the RequestTransformer behaviour with a callback module? WDYT?

The more I think about this, I don't see any real advantage to adding this option versus wrapping your Finch calls in a module that injects the tracing headers. If...

Thanks for clarifying @GregMefford! I do see your point, and I feel that the burden of keeping up with the Finch API could be minimized. I have been considering stabilizing...

Thanks for chiming in everyone! The point about the wrapper module suggestion turning this into essentially a global setting is definitely valid and turns me off of that idea. My...

No apology necessary @learc83, and congrats on your new baby! I am a new father myself (coming up on 1 year, so I'm starting to get the hang of it),...

Alright, I'm convinced! Glad that it has proven to be useful. I will clean things up and merge it soon.