Sampler Performance Testing

[nholbrook]

No description provided.

[codecov]

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   95.69%   95.69%           
=======================================
  Files          96       96           
  Lines        2878     2878           
=======================================
  Hits         2754     2754           
  Misses        124      124           

[nholbrook]

Current Benchmark Results:

Benchmark	Time	CPU	Iterations
BM_AlwaysOffSamplerConstruction	1.92 ns	1.67 ns	100000000
BM_AlwaysOnSamplerConstruction	1.89 ns	1.89 ns	74647736
BM_ParentOrElseSamplerConstruction	204 ns	203 ns	706505
BM_ProbabilitySamplerConstruction	372 ns	372 ns	387424
BM_AlwaysOffSamplerShouldSample	3.34 ns	3.34 ns	41949892
BM_AlwaysOnSamplerShouldSample	3.14 ns	3.14 ns	45300132
BM_ParentOrElseSamplerShouldSample	6.27 ns	6.27 ns	22260191
BM_ProbabilitySamplerShouldSample	5.38 ns	5.38 ns	26910475
BM_SpanCreation	657 ns	657 ns	207378
BM_NoopSpanCreation	56.1 ns	56.1 ns	2472011

[pyohannes]

This looks good, thanks. Results look good, the BM_NoopSpanCreation test will for sure be useful for further optimization work.

[pyohannes]

This is ready to be merged @reyang.

[reyang]

Looks like there is a Bazel update again - which caused the CI to fail. Not blocking this PR merge though.