Add .NET service names to macrobenchmarks#8490
Conversation
BenchmarksBenchmark execution time: 2026-04-21 11:32:01 Comparing candidate commit 3fa93fc in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 26 metrics, 0 unstable metrics, 87 known flaky benchmarks.
|
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8490) and master. ✅ No regressions detected - check the details below Full Metrics ComparisonFakeDbCommand
HttpMessageHandler
Comparison explanationExecution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:
Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard. Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph). Duration chartsFakeDbCommand (.NET Framework 4.8)gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (73ms) : 70, 75
master - mean (74ms) : 71, 78
section Bailout
This PR (8490) - mean (79ms) : 74, 84
master - mean (77ms) : 75, 79
section CallTarget+Inlining+NGEN
This PR (8490) - mean (1,082ms) : 1039, 1125
master - mean (1,079ms) : 1023, 1135
FakeDbCommand (.NET Core 3.1)gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (115ms) : 111, 118
master - mean (116ms) : 111, 122
section Bailout
This PR (8490) - mean (119ms) : 113, 126
master - mean (116ms) : 113, 119
section CallTarget+Inlining+NGEN
This PR (8490) - mean (799ms) : 775, 822
master - mean (800ms) : 779, 821
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (104ms) : 99, 110
master - mean (101ms) : 98, 104
section Bailout
This PR (8490) - mean (103ms) : 100, 106
master - mean (105ms) : 98, 111
section CallTarget+Inlining+NGEN
This PR (8490) - mean (947ms) : 906, 988
master - mean (940ms) : 907, 973
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (101ms) : 97, 105
master - mean (100ms) : 95, 105
section Bailout
This PR (8490) - mean (105ms) : 98, 111
master - mean (100ms) : 98, 103
section CallTarget+Inlining+NGEN
This PR (8490) - mean (827ms) : 787, 866
master - mean (830ms) : 784, 876
HttpMessageHandler (.NET Framework 4.8)gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (203ms) : 194, 211
master - mean (201ms) : 193, 209
section Bailout
This PR (8490) - mean (208ms) : 198, 218
master - mean (206ms) : 199, 213
section CallTarget+Inlining+NGEN
This PR (8490) - mean (1,202ms) : 1146, 1259
master - mean (1,205ms) : 1144, 1266
HttpMessageHandler (.NET Core 3.1)gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (294ms) : 281, 307
master - mean (290ms) : 279, 302
section Bailout
This PR (8490) - mean (292ms) : 280, 305
master - mean (291ms) : 280, 303
section CallTarget+Inlining+NGEN
This PR (8490) - mean (980ms) : 949, 1011
master - mean (965ms) : 937, 993
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (287ms) : 271, 304
master - mean (284ms) : 274, 294
section Bailout
This PR (8490) - mean (288ms) : 273, 303
master - mean (285ms) : 275, 296
section CallTarget+Inlining+NGEN
This PR (8490) - mean (1,167ms) : 1126, 1208
master - mean (1,158ms) : 1122, 1194
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8490) - mean (286ms) : 272, 301
master - mean (280ms) : 271, 290
section Bailout
This PR (8490) - mean (287ms) : 271, 302
master - mean (282ms) : 272, 291
section CallTarget+Inlining+NGEN
This PR (8490) - mean (1,058ms) : 980, 1136
master - mean (1,051ms) : 971, 1131
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Summary of changes
Adds
DD_SERVICEto each of the benchmark jobsReason for change
Lets us compare between scenarios in telemetry metrics
Implementation details
Manually added
DD_SERVICEto each jobTest coverage
I'll trigger a manual run for the branch to confirm we can see this data as expected
Other details
We may want to make the service name dynamic in the future, so we can isolate these to a specific commit, but I think that's not critical atm, given that these mostly only run on master, so we should generally be able to distinguish based on start time (except when there are multiple commits to master)