Jay White
Jay White
>This PR is not meant to close #229 but serves as initial support. We should use this as a stepping stone and, after discussions arising from the code, decide how...
It appears that opentelemetry-jaeger is deprecated. I imagine that removing the use of this would fix the issue.
If I'm not mistaken, these are only used in postprocessing, which is why those queries still work: they don't use postprocessing. Regardless, these should be filled out.
I believe it is open. Some of the parts may be partially completed. @stuarttimwhite can confirm
@tmahmood I added a quick review.
@ssddOnTop are you waiting on us here? I don't see activity on this.
/bounty $100 I will give at least $100 for cutting the total test time in half. In other words, on my machine: * ~7.5 min -> $100 * ~3.75 min...
Let me clarify a couple things that hopefully make this less frustrating. * The motivation behind this issue is the fact that CI takes a long time to run. The...
@zelosleone Responding to each of your points: 1 - If `opt-level = 3` will make the CI faster, awesome. Put in a 2 line PR. 2 - If switching to...