-
Notifications
You must be signed in to change notification settings - Fork 848
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Benchmark usage of the SpanContext #792
Comments
@jkwatson can u provide more details about it? I would like to help with it |
@DotSpy I think part of this issue is to dig into the SpanContext and figure out what might need some benchmarking. I haven't had time to dig into it myself. Are you up for doing some of that digging and making some benchmark recommendations that could be built? |
Yeap, i will dig and ask questions during investigation, if all go right i will be able to switch to another tasks with benchmarks. Do we have restrictions for frameworks that i can use for it? |
Also i saw some benchmarks in opentelemetry-js project, i will ask guys there |
There's a pending PR to use JMH for the benchmarking (#678). I think we should stick with that. |
@jkwatson Assign me to this one please |
I did main PR benchmarks for context, for beta i think it's okay and we need to find some hardware on which we can run it to make measurements stick to it. So after we will be able to improve performance and write more specific benchmarks for context. @jkwatson what do u think is it possible to find such hardware? |
I think we'd need to lean on our friends in the CNCF to see if we can get a benchmarking platform. Not sure who can drive this. @bogdandrutu maybe? |
Is there anything further we'd like to benchmark around the SpanContext itself, @carlosalberto ? |
No description provided.
The text was updated successfully, but these errors were encountered: