-
Notifications
You must be signed in to change notification settings - Fork 160
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
Add TEA Benchmarks #2438
Add TEA Benchmarks #2438
Conversation
f68c3c5
to
e7ecc45
Compare
BenchmarksBenchmark execution time: 2024-01-10 14:18:30 Comparing candidate commit e7ecc45 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 38 metrics, 51 unstable metrics. scenario:ContextPropagationBench/benchExtractHeaders128Bit
|
b4c6c92
to
ef3088f
Compare
239d3fc
to
a7aaa2d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me, let's merge this :-)
a7aaa2d
to
b590053
Compare
Description
(TEA) Benchmarks request startup & spindown using Google benchmarks.
These benchmarks can be run using
make benchmarks_tea
from the root of the repository. Sample results are provided below.This PR adds GBench as a submodule rather than installing it globally in the buster images. The choice of GBench over Catch2 was made for the following reasons:
On the benchmarking-platform side, I'll cherry-pick these two commits: commit1, commit2. It adds missing requirements to build tea and analyze results.
Sample JSON Results
Reviewer checklist