Skip to content

Comments

David.ogbureke/eventbridge 1mb payload limit#8229

Draft
Dogbu-cyber wants to merge 2 commits intomasterfrom
david.ogbureke/eventbridge-1mb-payload-limit
Draft

David.ogbureke/eventbridge 1mb payload limit#8229
Dogbu-cyber wants to merge 2 commits intomasterfrom
david.ogbureke/eventbridge-1mb-payload-limit

Conversation

@Dogbu-cyber
Copy link

Summary of changes

Reason for change

Implementation details

Test coverage

Other details

@pr-commenter
Copy link

pr-commenter bot commented Feb 20, 2026

Benchmarks

Benchmark execution time: 2026-02-20 18:44:10

Comparing candidate commit de24b54 in PR branch david.ogbureke/eventbridge-1mb-payload-limit with baseline commit 7ff6b28 in branch master.

Found 4 performance improvements and 10 performance regressions! Performance is the same for 166 metrics, 12 unstable metrics.

scenario:Benchmarks.Trace.ActivityBenchmark.StartStopWithChild net472

  • 🟥 throughput [-4926.126op/s; -4520.629op/s] or [-5.870%; -5.387%]

scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces net472

  • 🟥 throughput [-55.240op/s; -52.706op/s] or [-9.345%; -8.916%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleSimpleBody net6.0

  • 🟩 execution_time [-26.078ms; -19.803ms] or [-11.732%; -8.909%]

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

  • 🟥 execution_time [+25.273ms; +29.706ms] or [+12.918%; +15.184%]

scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces net472

  • 🟩 throughput [+59.652op/s; +160.213op/s] or [+5.812%; +15.609%]

scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces net6.0

  • 🟥 execution_time [+9.942ms; +15.045ms] or [+6.294%; +9.525%]

scenario:Benchmarks.Trace.GraphQLBenchmark.ExecuteAsync net6.0

  • 🟥 execution_time [+10.439ms; +11.547ms] or [+5.339%; +5.905%]

scenario:Benchmarks.Trace.ILoggerBenchmark.EnrichedLog net6.0

  • 🟥 execution_time [+13.511ms; +17.197ms] or [+6.844%; +8.711%]

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

  • 🟥 throughput [-435.955op/s; -238.664op/s] or [-20.003%; -10.951%]

scenario:Benchmarks.Trace.Log4netBenchmark.EnrichedLog netcoreapp3.1

  • 🟥 execution_time [+13.064ms; +14.090ms] or [+6.919%; +7.463%]

scenario:Benchmarks.Trace.RedisBenchmark.SendReceive netcoreapp3.1

  • 🟥 throughput [-39374.679op/s; -29625.294op/s] or [-9.150%; -6.885%]

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

  • 🟩 execution_time [-60.937ms; -58.432ms] or [-41.099%; -39.410%]

scenario:Benchmarks.Trace.SpanBenchmark.StartFinishSpan netcoreapp3.1

  • 🟩 execution_time [-21.649ms; -16.496ms] or [-10.106%; -7.700%]

scenario:Benchmarks.Trace.TraceAnnotationsBenchmark.RunOnMethodBegin netcoreapp3.1

  • 🟥 execution_time [+10.154ms; +14.363ms] or [+5.141%; +7.272%]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant