dd-trace-php icon indicating copy to clipboard operation
dd-trace-php copied to clipboard

[WIP] Add support for processor overrides

Open Anilm3 opened this issue 7 months ago • 2 comments
trafficstars

Description

Reviewer checklist

  • [ ] Test coverage seems ok.
  • [ ] Appropriate labels assigned.

Anilm3 avatar Apr 15 '25 09:04 Anilm3

Benchmarks [ tracer ]

Benchmark execution time: 2025-04-17 12:20:35

Comparing candidate commit 6fcd0449851c9c77fc9fc4063830a5b4b69df642 in PR branch anilm3/processor-overrides with baseline commit b2a1ef59c8ed5947d2cbc099ee5174eb534cf630 in branch master.

Found 1 performance improvements and 1 performance regressions! Performance is the same for 176 metrics, 0 unstable metrics.

scenario:PDOBench/benchPDOBaseline

  • 🟥 execution_time [+11.930µs; +16.182µs] or [+6.409%; +8.694%]

scenario:TraceFlushBench/benchFlushTrace

  • 🟩 execution_time [-1000.000ns; -1000.000ns] or [-50.000%; -50.000%]

pr-commenter[bot] avatar Apr 15 '25 09:04 pr-commenter[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.38%. Comparing base (b2a1ef5) to head (6fcd044).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3213      +/-   ##
============================================
- Coverage     79.30%   76.38%   -2.93%     
  Complexity     2943     2943              
============================================
  Files           117      144      +27     
  Lines         11603    16046    +4443     
  Branches          0     1107    +1107     
============================================
+ Hits           9202    12257    +3055     
- Misses         2401     3214     +813     
- Partials          0      575     +575     
Flag Coverage Δ
appsec-extension 68.80% <ø> (?)
tracer-php 79.28% <ø> (-0.02%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

see 28 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b2a1ef5...6fcd044. Read the comment docs.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Apr 15 '25 09:04 codecov-commenter