Closed bwoebi closed 3 weeks ago
Attention: Patch coverage is 72.72727%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 77.96%. Comparing base (
c5f2dcb
) to head (f91f0db
).
Benchmark execution time: 2024-06-08 16:21:37
Comparing candidate commit f91f0dbcc4b1294acb0e269b5bf6ae26a891782f in PR branch bob/fix-utf8-msgpack
with baseline commit c5f2dcb3faec360227837310221040ce6208e082 in branch master
.
Found 3 performance improvements and 5 performance regressions! Performance is the same for 170 metrics, 0 unstable metrics.
execution_time
[+74.329µs; +77.611µs] or [+50.151%; +52.366%]execution_time
[+75.011µs; +76.889µs] or [+50.639%; +51.906%]execution_time
[+14.951µs; +17.647µs] or [+8.600%; +10.151%]execution_time
[+15.505µs; +18.772µs] or [+5.613%; +6.796%]execution_time
[+5.792µs; +14.535µs] or [+2.007%; +5.037%]execution_time
[-457.792ns; -184.608ns] or [-6.588%; -2.657%]execution_time
[-398.199ns; -152.601ns] or [-5.782%; -2.216%]execution_time
[-534.659ns; -284.541ns] or [-7.689%; -4.092%]
Ensure no utf8 is contained in msgpack strings.
The agent would properly sanitize this away, but the sidecar is not. All traces containing a single invalid utf-8 sequence were dropped.