Pin floating pre-release versions of SQLitePCLRaw.* dependencies in samples#8390
Pin floating pre-release versions of SQLitePCLRaw.* dependencies in samples#8390
Conversation
BenchmarksBenchmark execution time: 2026-03-30 20:58:30 Comparing candidate commit 61867e5 in PR branch Found 15 performance improvements and 12 performance regressions! Performance is the same for 244 metrics, 17 unstable metrics.
|
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8390) 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 (8390) - mean (72ms) : 70, 74
master - mean (72ms) : 70, 74
section Bailout
This PR (8390) - mean (76ms) : 74, 78
master - mean (76ms) : 74, 78
section CallTarget+Inlining+NGEN
This PR (8390) - mean (1,080ms) : 1030, 1131
master - mean (1,078ms) : 1030, 1127
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 (8390) - mean (111ms) : 108, 114
master - mean (112ms) : 108, 116
section Bailout
This PR (8390) - mean (113ms) : 111, 115
master - mean (113ms) : 110, 116
section CallTarget+Inlining+NGEN
This PR (8390) - mean (799ms) : 776, 821
master - mean (795ms) : 778, 812
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8390) - mean (99ms) : 96, 102
master - mean (100ms) : 95, 105
section Bailout
This PR (8390) - mean (100ms) : 98, 102
master - mean (100ms) : 98, 103
section CallTarget+Inlining+NGEN
This PR (8390) - mean (949ms) : 913, 986
master - mean (948ms) : 907, 989
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8390) - mean (99ms) : 96, 102
master - mean (98ms) : 95, 102
section Bailout
This PR (8390) - mean (100ms) : 98, 102
master - mean (100ms) : 97, 102
section CallTarget+Inlining+NGEN
This PR (8390) - mean (828ms) : 793, 864
master - mean (829ms) : 795, 862
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 (8390) - mean (209ms) : 204, 214
master - mean (214ms) : 209, 219
section Bailout
This PR (8390) - mean (214ms) : 209, 218
master - mean (217ms) : 211, 223
section CallTarget+Inlining+NGEN
This PR (8390) - mean (1,234ms) : 1174, 1295
master - mean (1,258ms) : 1211, 1305
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 (8390) - mean (303ms) : 297, 310
master - mean (310ms) : 303, 316
section Bailout
This PR (8390) - mean (304ms) : 296, 313
master - mean (312ms) : 305, 320
section CallTarget+Inlining+NGEN
This PR (8390) - mean (1,021ms) : 993, 1049
master - mean (1,035ms) : 1001, 1070
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8390) - mean (295ms) : 289, 301
master - mean (305ms) : 296, 315
section Bailout
This PR (8390) - mean (297ms) : 288, 306
master - mean (304ms) : 298, 310
section CallTarget+Inlining+NGEN
This PR (8390) - mean (1,192ms) : 1158, 1227
master - mean (1,205ms) : 1169, 1242
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8390) - mean (294ms) : 286, 301
master - mean (301ms) : 291, 311
section Bailout
This PR (8390) - mean (294ms) : 286, 302
master - mean (302ms) : 294, 310
section CallTarget+Inlining+NGEN
This PR (8390) - mean (1,114ms) : 999, 1229
master - mean (1,127ms) : 1013, 1241
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Summary of changes
Pins two different projects that were using floating versions of PackageReferences to
SQLitePCLRaw.bundle_e_sqlite3andSQLitePCLRaw.core.These appeared to link to these versions, I bumped them to be stable
Reason for change
We don't want any floating dependencies
Implementation details
Searched here: https://github.com/search?q=repo%3ADataDog%2Fdd-trace-dotnet%20%2FVersion%3D%22%5B%5E%22%5D*%5C*%2F&type=code
Test coverage
This is the test, hopefully this works
Other details