Skip to content

Commit e62315b

Browse files
authored
fix benchmark code (#8104)
## Summary of changes signature changed in #7871, and since the benchmark CI step was broken for another reason, I missed that. ## Reason for change ## Implementation details ## Test coverage ## Other details <!-- Fixes #{issue} --> <!-- ⚠️ Note: Where possible, please obtain 2 approvals prior to merging. Unless CODEOWNERS specifies otherwise, for external teams it is typically best to have one review from a team member, and one review from apm-dotnet. Trivial changes do not require 2 reviews. MergeQueue is NOT enabled in this repository. If you have write access to the repo, the PR has 1-2 approvals (see above), and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #apm-dotnet channel in Slack. -->
1 parent 993aaba commit e62315b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tracer/test/benchmarks/Benchmarks.Trace/AgentWriterBenchmark.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,14 @@ public void GlobalSetup()
5353

5454
var api = new Api(
5555
new FakeApiRequestFactory(settings.Manager.InitialExporterSettings.AgentUri),
56-
new StatsdManager(settings, (_, _) => null!),
56+
new StatsdManager(settings, (_, _, _) => null!),
5757
ContainerMetadata.Instance,
5858
updateSampleRates: null,
5959
updateConfigState: null,
6060
partialFlushEnabled: false,
6161
healthMetricsEnabled: false);
6262

63-
var noOpStatsd = new StatsdManager(settings, (_, _) => null);
63+
var noOpStatsd = new StatsdManager(settings, (_, _, _) => null);
6464
var noopApi = new NullApi();
6565
_agentWriter = new AgentWriter(api, statsAggregator: null, statsd: noOpStatsd, automaticFlush: false);
6666
_agentWriterNoOpFlush = new AgentWriter(noopApi, statsAggregator: null, statsd: noOpStatsd, automaticFlush: false);

0 commit comments

Comments
 (0)