Douglas Q Hawkins
Douglas Q Hawkins
I understand the desire to short-circuit the calls, but I'd like to understand why we've chosen this approach to do it. I see that the callback only takes Strings. That's...
I'm seeing a number of thread safety and overhead issues in these pull requests. I think it would be good to discuss the threading requirements of objects in the tracer.
We don't have any immediate plans to add support for Camel. However, we did add support for OpenTelemetry APIs recently.
I'll admit that the lack of count support partially is down to laziness so far. I'm happy to try implementing it and then we can measure the impact. I'm not...
> It seems it'd be relatively straightforward to keep track of the count in the map and avoid the O(n) price of computing it (or checking for emptiness), was it...
I believe getTag("error.type") returns a String not a Throwable instance. It should contain the name of the exception type. So the cast to Throwable there is likely raising a ClassCastException...
OpenTelemetry doesn't really have an equivalent to Datadog's operation name. OpenTelemetry's span name is closest to Datadog's resource name. @PerfectSlayer can probably provide a bit more detail.
Thank you for the report, we are looking into the increased CPU usage.