Skip to content

Add support for Microsoft.Data.SqlClient 7.x.x#8346

Merged
andrewlock merged 2 commits into
masterfrom
andrew/bump-microsoft-sqlclient
Mar 20, 2026
Merged

Add support for Microsoft.Data.SqlClient 7.x.x#8346
andrewlock merged 2 commits into
masterfrom
andrew/bump-microsoft-sqlclient

Conversation

@andrewlock

@andrewlock andrewlock commented Mar 19, 2026

Copy link
Copy Markdown
Member

Summary of changes

Adds support for Microsoft.Data.SqlClient 7.x.x

Reason for change

We want to support the latest packages

Implementation details

  • Bump the version in SqlClientDefinitions.cs
  • .\tracer\build.ps1 CompileManagedSrc
  • .\tracer\build.ps1 GeneratePackageVersions -IncludePackages Microsoft.Data.SqlClient --skip

Test coverage

This is the test

@andrewlock
andrewlock requested review from a team as code owners March 19, 2026 19:02
@andrewlock andrewlock added area:automatic-instrumentation Automatic instrumentation managed C# code (Datadog.Trace.ClrProfiler.Managed) area:dependabot dependabot updates area:integrations labels Mar 19, 2026
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Mar 19, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 79b170fdca

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ...r/src/Datadog.Trace/ClrProfiler/AutoInstrumentation/AdoNet/SqlClient/SqlClientDefinitions.cs
@pr-commenter

pr-commenter Bot commented Mar 19, 2026 •
edited
Loading

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-03-20 14:33:55

Comparing candidate commit adea202 in PR branch andrew/bump-microsoft-sqlclient with baseline commit b0dc7b1 in branch master.

Found 4 performance improvements and 7 performance regressions! Performance is the same for 256 metrics, 21 unstable metrics.

Explanation

This is an A/B test comparing a candidate commit's performance against that of a baseline commit. Performance changes are noted in the tables below as:

  • 🟩 = significantly better candidate vs. baseline
  • 🟥 = significantly worse candidate vs. baseline

We compute a confidence interval (CI) over the relative difference of means between metrics from the candidate and baseline commits, considering the baseline as the reference.

If the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD), the change is considered significant.

Feel free to reach out to #apm-benchmarking-platform on Slack if you have any questions.

More details about the CI and significant changes

You can imagine this CI as a range of values that is likely to contain the true difference of means between the candidate and baseline commits.

CIs of the difference of means are often centered around 0%, because often changes are not that big:

---------------------------------(------|---^--------)-------------------------------->
                              -0.6%    0%  0.3%     +1.2%
                                 |          |        |
         lower bound of the CI --'          |        |
sample mean (center of the CI) -------------'        |
         upper bound of the CI ----------------------'

As described above, a change is considered significant if the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD).

For instance, for an execution time metric, this confidence interval indicates a significantly worse performance:

----------------------------------------|---------|---(---------^---------)---------->
                                       0%        1%  1.3%      2.2%      3.1%
                                                  |   |         |         |
       significant impact threshold --------------'   |         |         |
                      lower bound of CI --------------'         |         |
       sample mean (center of the CI) --------------------------'         |
                      upper bound of CI ----------------------------------'

scenario:Benchmarks.Trace.ActivityBenchmark.StartStopWithChild netcoreapp3.1

  • 🟥 execution_time [+113.379ms; +115.543ms] or [+130.764%; +133.260%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleSimpleBody netcoreapp3.1

  • 🟩 execution_time [-23.517ms; -17.310ms] or [-10.715%; -7.887%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.ObjectExtractorSimpleBody netcoreapp3.1

  • 🟩 execution_time [-23.143ms; -16.853ms] or [-10.738%; -7.820%]
  • 🟩 throughput [+245213.968op/s; +322283.536op/s] or [+10.935%; +14.371%]

scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces net472

  • 🟥 execution_time [+15.997ms; +21.885ms] or [+7.950%; +10.877%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OriginalCharSlice net472

  • 🟥 execution_time [+164.518µs; +173.575µs] or [+6.359%; +6.709%]
  • 🟥 throughput [-24.340op/s; -23.070op/s] or [-6.297%; -5.969%]

scenario:Benchmarks.Trace.ElasticsearchBenchmark.CallElasticsearch net6.0

  • 🟥 throughput [-40653.502op/s; -39467.548op/s] or [-6.131%; -5.953%]

scenario:Benchmarks.Trace.Iast.StringAspectsBenchmark.StringConcatAspectBenchmark netcoreapp3.1

  • 🟥 allocated_mem [+18.130KB; +18.156KB] or [+7.053%; +7.063%]

scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore net6.0

  • 🟩 execution_time [-15.729ms; -7.496ms] or [-15.768%; -7.514%]

scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore netcoreapp3.1

  • 🟥 throughput [-16360941.105op/s; -15434348.919op/s] or [-6.788%; -6.403%]

@andrewlock
andrewlock force-pushed the andrew/bump-microsoft-sqlclient branch from 79b170f to 703bca8 Compare March 20, 2026 07:58
@dd-trace-dotnet-ci-bot

dd-trace-dotnet-ci-bot Bot commented Mar 20, 2026 •
edited
Loading

Copy link
Copy Markdown

Execution-Time Benchmarks Report ⏱️

Execution-time results for samples comparing This PR (8346) and master.

✅ No regressions detected - check the details below

Full Metrics Comparison

FakeDbCommand

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration69.31 ± (69.54 - 69.99) ms68.85 ± (68.82 - 69.07) ms-0.7%✅
.NET Framework 4.8 - Bailout
duration73.09 ± (73.02 - 73.28) ms72.81 ± (72.73 - 73.00) ms-0.4%✅
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1044.03 ± (1046.70 - 1054.47) ms1038.16 ± (1040.87 - 1046.28) ms-0.6%✅
.NET Core 3.1 - Baseline
process.internal_duration_ms21.90 ± (21.87 - 21.94) ms21.96 ± (21.93 - 22.00) ms+0.3%✅⬆️
process.time_to_main_ms79.85 ± (79.68 - 80.03) ms79.86 ± (79.71 - 80.01) ms+0.0%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.89 ± (10.89 - 10.89) MB10.91 ± (10.91 - 10.92) MB+0.2%✅⬆️
runtime.dotnet.threads.count12 ± (12 - 12)12 ± (12 - 12)+0.0%✅
.NET Core 3.1 - Bailout
process.internal_duration_ms21.88 ± (21.85 - 21.91) ms21.87 ± (21.84 - 21.89) ms-0.1%✅
process.time_to_main_ms81.31 ± (81.16 - 81.45) ms80.96 ± (80.81 - 81.11) ms-0.4%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.94 ± (10.93 - 10.94) MB10.93 ± (10.93 - 10.94) MB-0.0%✅
runtime.dotnet.threads.count13 ± (13 - 13)13 ± (13 - 13)+0.0%✅
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms220.38 ± (219.28 - 221.48) ms219.61 ± (218.46 - 220.76) ms-0.3%✅
process.time_to_main_ms472.68 ± (472.05 - 473.32) ms473.25 ± (472.68 - 473.81) ms+0.1%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed47.24 ± (47.21 - 47.27) MB47.23 ± (47.20 - 47.25) MB-0.0%✅
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)-1.0%✅
.NET 6 - Baseline
process.internal_duration_ms20.78 ± (20.75 - 20.81) ms20.73 ± (20.71 - 20.76) ms-0.2%✅
process.time_to_main_ms69.91 ± (69.76 - 70.07) ms69.67 ± (69.54 - 69.81) ms-0.3%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.62 ± (10.62 - 10.62) MB10.64 ± (10.63 - 10.64) MB+0.2%✅⬆️
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%✅
.NET 6 - Bailout
process.internal_duration_ms20.52 ± (20.50 - 20.54) ms20.63 ± (20.60 - 20.66) ms+0.5%✅⬆️
process.time_to_main_ms70.16 ± (70.07 - 70.26) ms70.31 ± (70.21 - 70.40) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.73 ± (10.73 - 10.74) MB10.75 ± (10.74 - 10.75) MB+0.1%✅⬆️
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%✅
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms206.33 ± (204.11 - 208.56) ms199.98 ± (198.02 - 201.94) ms-3.1%✅
process.time_to_main_ms473.56 ± (472.71 - 474.41) ms473.27 ± (472.49 - 474.04) ms-0.1%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed49.03 ± (48.97 - 49.10) MB48.94 ± (48.88 - 49.01) MB-0.2%✅
runtime.dotnet.threads.count29 ± (29 - 29)29 ± (29 - 29)-0.1%✅
.NET 8 - Baseline
process.internal_duration_ms18.85 ± (18.82 - 18.88) ms18.95 ± (18.92 - 18.97) ms+0.5%✅⬆️
process.time_to_main_ms68.63 ± (68.48 - 68.77) ms68.58 ± (68.46 - 68.69) ms-0.1%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed7.69 ± (7.68 - 7.69) MB7.68 ± (7.67 - 7.69) MB-0.1%✅
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%✅
.NET 8 - Bailout
process.internal_duration_ms18.88 ± (18.86 - 18.91) ms19.04 ± (19.00 - 19.07) ms+0.8%✅⬆️
process.time_to_main_ms69.67 ± (69.55 - 69.79) ms70.01 ± (69.89 - 70.14) ms+0.5%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed7.74 ± (7.73 - 7.75) MB7.73 ± (7.72 - 7.74) MB-0.1%✅
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%✅
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms157.63 ± (156.77 - 158.48) ms155.95 ± (155.05 - 156.86) ms-1.1%✅
process.time_to_main_ms451.64 ± (451.12 - 452.16) ms450.07 ± (449.56 - 450.57) ms-0.3%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed36.53 ± (36.51 - 36.55) MB36.57 ± (36.56 - 36.59) MB+0.1%✅⬆️
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)+0.1%✅⬆️

HttpMessageHandler

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration195.58 ± (195.53 - 196.34) ms195.05 ± (194.93 - 195.77) ms-0.3%✅
.NET Framework 4.8 - Bailout
duration197.34 ± (197.14 - 197.92) ms197.46 ± (197.28 - 197.86) ms+0.1%✅⬆️
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1143.98 ± (1144.38 - 1151.56) ms1152.96 ± (1155.27 - 1162.63) ms+0.8%✅⬆️
.NET Core 3.1 - Baseline
process.internal_duration_ms188.27 ± (187.94 - 188.59) ms188.60 ± (188.18 - 189.02) ms+0.2%✅⬆️
process.time_to_main_ms80.90 ± (80.73 - 81.08) ms81.64 ± (81.41 - 81.86) ms+0.9%✅⬆️
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed16.14 ± (16.10 - 16.17) MB16.11 ± (16.08 - 16.13) MB-0.2%✅
runtime.dotnet.threads.count20 ± (19 - 20)20 ± (20 - 20)+0.2%✅⬆️
.NET Core 3.1 - Bailout
process.internal_duration_ms188.05 ± (187.62 - 188.48) ms189.40 ± (188.93 - 189.87) ms+0.7%✅⬆️
process.time_to_main_ms82.62 ± (82.44 - 82.80) ms83.15 ± (82.97 - 83.32) ms+0.6%✅⬆️
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed16.09 ± (16.07 - 16.12) MB16.13 ± (16.09 - 16.18) MB+0.3%✅⬆️
runtime.dotnet.threads.count21 ± (20 - 21)21 ± (20 - 21)+0.2%✅⬆️
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms397.34 ± (395.60 - 399.08) ms395.51 ± (393.64 - 397.37) ms-0.5%✅
process.time_to_main_ms474.27 ± (473.58 - 474.95) ms474.88 ± (474.26 - 475.50) ms+0.1%✅⬆️
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed57.93 ± (57.78 - 58.08) MB57.83 ± (57.67 - 57.99) MB-0.2%✅
runtime.dotnet.threads.count30 ± (30 - 30)29 ± (29 - 30)-0.6%✅
.NET 6 - Baseline
process.internal_duration_ms192.60 ± (192.19 - 193.00) ms193.68 ± (193.30 - 194.06) ms+0.6%✅⬆️
process.time_to_main_ms70.45 ± (70.24 - 70.66) ms70.57 ± (70.43 - 70.72) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed16.15 ± (16.02 - 16.28) MB16.40 ± (16.37 - 16.44) MB+1.6%✅⬆️
runtime.dotnet.threads.count18 ± (18 - 19)19 ± (19 - 19)+4.2%✅⬆️
.NET 6 - Bailout
process.internal_duration_ms191.69 ± (191.36 - 192.03) ms193.86 ± (193.35 - 194.37) ms+1.1%✅⬆️
process.time_to_main_ms71.18 ± (71.06 - 71.30) ms71.61 ± (71.44 - 71.78) ms+0.6%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed16.17 ± (16.04 - 16.31) MB16.44 ± (16.41 - 16.47) MB+1.6%✅⬆️
runtime.dotnet.threads.count20 ± (20 - 20)20 ± (20 - 20)+1.7%✅⬆️
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms428.23 ± (426.42 - 430.04) ms430.64 ± (428.96 - 432.32) ms+0.6%✅⬆️
process.time_to_main_ms476.18 ± (475.42 - 476.94) ms477.35 ± (476.47 - 478.22) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed60.22 ± (60.15 - 60.29) MB60.20 ± (60.12 - 60.28) MB-0.0%✅
runtime.dotnet.threads.count30 ± (30 - 30)30 ± (30 - 30)+0.2%✅⬆️
.NET 8 - Baseline
process.internal_duration_ms190.84 ± (190.41 - 191.27) ms191.45 ± (191.03 - 191.87) ms+0.3%✅⬆️
process.time_to_main_ms70.21 ± (70.00 - 70.43) ms70.28 ± (70.08 - 70.49) ms+0.1%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed11.77 ± (11.74 - 11.79) MB11.77 ± (11.74 - 11.79) MB+0.0%✅⬆️
runtime.dotnet.threads.count18 ± (18 - 18)18 ± (18 - 18)+0.2%✅⬆️
.NET 8 - Bailout
process.internal_duration_ms190.29 ± (189.98 - 190.61) ms190.80 ± (190.41 - 191.19) ms+0.3%✅⬆️
process.time_to_main_ms70.94 ± (70.81 - 71.06) ms71.31 ± (71.16 - 71.45) ms+0.5%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed11.85 ± (11.82 - 11.88) MB11.78 ± (11.75 - 11.81) MB-0.6%✅
runtime.dotnet.threads.count19 ± (19 - 19)19 ± (19 - 19)-0.6%✅
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms351.91 ± (350.56 - 353.26) ms356.31 ± (354.93 - 357.68) ms+1.2%✅⬆️
process.time_to_main_ms452.69 ± (452.09 - 453.29) ms454.39 ± (453.78 - 454.99) ms+0.4%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed48.41 ± (48.36 - 48.45) MB48.46 ± (48.42 - 48.50) MB+0.1%✅⬆️
runtime.dotnet.threads.count30 ± (30 - 30)30 ± (30 - 30)-0.1%✅
Comparison explanation

Execution-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:

  • Welch test with statistical test for significance of 5%
  • Only results indicating a difference greater than 5% and 5 ms are considered.

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 charts
FakeDbCommand (.NET Framework 4.8) 
gantt
    title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8346) - mean (69ms)  : 67, 71
    master - mean (70ms)  : 66, 73

    section Bailout
    This PR (8346) - mean (73ms)  : 72, 74
    master - mean (73ms)  : 72, 74

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (1,044ms)  : 1004, 1083
    master - mean (1,051ms)  : 995, 1106
Loading
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 (8346) - mean (108ms)  : 105, 111
    master - mean (107ms)  : 104, 111

    section Bailout
    This PR (8346) - mean (108ms)  : 106, 110
    master - mean (109ms)  : 107, 110

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (728ms)  : 706, 750
    master - mean (730ms)  : 711, 750
Loading
FakeDbCommand (.NET 6) 
gantt
    title Execution time (ms) FakeDbCommand (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8346) - mean (96ms)  : 94, 98
    master - mean (96ms)  : 93, 99

    section Bailout
    This PR (8346) - mean (96ms)  : 94, 98
    master - mean (96ms)  : 94, 98

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (703ms)  : 658, 748
    master - mean (709ms)  : 668, 751
Loading
FakeDbCommand (.NET 8) 
gantt
    title Execution time (ms) FakeDbCommand (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8346) - mean (94ms)  : 91, 97
    master - mean (94ms)  : 92, 97

    section Bailout
    This PR (8346) - mean (96ms)  : 94, 98
    master - mean (95ms)  : 93, 97

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (638ms)  : 615, 661
    master - mean (641ms)  : 615, 667
Loading
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 (8346) - mean (195ms)  : 191, 199
    master - mean (196ms)  : 191, 201

    section Bailout
    This PR (8346) - mean (198ms)  : 195, 200
    master - mean (198ms)  : 194, 201

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (1,159ms)  : 1105, 1212
    master - mean (1,148ms)  : 1097, 1199
Loading
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 (8346) - mean (279ms)  : 273, 284
    master - mean (277ms)  : 273, 282

    section Bailout
    This PR (8346) - mean (281ms)  : 274, 288
    master - mean (279ms)  : 274, 284

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (903ms)  : 869, 936
    master - mean (903ms)  : 876, 929
Loading
HttpMessageHandler (.NET 6) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8346) - mean (272ms)  : 266, 278
    master - mean (272ms)  : 266, 277

    section Bailout
    This PR (8346) - mean (273ms)  : 267, 280
    master - mean (271ms)  : 267, 275

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (940ms)  : 915, 965
    master - mean (936ms)  : 909, 963
Loading
HttpMessageHandler (.NET 8) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8346) - mean (272ms)  : 263, 281
    master - mean (271ms)  : 263, 278

    section Bailout
    This PR (8346) - mean (272ms)  : 266, 277
    master - mean (271ms)  : 266, 275

    section CallTarget+Inlining+NGEN
    This PR (8346) - mean (841ms)  : 813, 868
    master - mean (835ms)  : 814, 855
Loading

@andrewlock
andrewlock force-pushed the andrew/bump-microsoft-sqlclient branch from 703bca8 to adea202 Compare March 20, 2026 13:52
@bouwkast bouwkast mentioned this pull request Mar 20, 2026
Closed
bouwkast
bouwkast approved these changes Mar 20, 2026
View reviewed changes
Comment thread tracer/build/PackageVersionsLatestSpecific.g.props
@andrewlock
andrewlock enabled auto-merge (squash) March 20, 2026 15:07
@andrewlock
andrewlock merged commit 5b27cd2 into master Mar 20, 2026
139 checks passed
@andrewlock
andrewlock deleted the andrew/bump-microsoft-sqlclient branch March 20, 2026 16:08
@github-actions github-actions Bot added this to the vNext-v3 milestone Mar 20, 2026
@andrewlock andrewlock mentioned this pull request Mar 20, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments
@bouwkast bouwkast bouwkast approved these changes

Assignees

No one assigned

Labels

area:automatic-instrumentation Automatic instrumentation managed C# code (Datadog.Trace.ClrProfiler.Managed) area:dependabot dependabot updates area:integrations

Projects

None yet

Milestone

3.40.0

Development

Successfully merging this pull request may close these issues.

2 participants

@andrewlock @bouwkast