Skip to content

Conversation

morrisonlevi
Copy link
Collaborator

Description

Evaluate using reqwest directly instead of using libdatadog's profiling exporter. Specifically, look at the Unix Domain Sockets (UDS) support.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@morrisonlevi morrisonlevi added ☠️ do-not-merge/WIP profiling Relates to the Continuous Profiler labels Jul 11, 2025
@pr-commenter
Copy link

pr-commenter bot commented Jul 11, 2025

Benchmarks [ profiler ]

Benchmark execution time: 2025-07-15 22:09:05

Comparing candidate commit ba8f818 in PR branch levi/reqwest with baseline commit 60f33e9 in branch levi/cargo-deps.

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

scenario:php-profiler-timeline-memory-with-profiler-and-timeline

  • 🟩 max_rss_usage [-1.444MB; -0.706MB] or [-4.530%; -2.217%]

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.72%. Comparing base (60f33e9) to head (25d0c84).

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##           levi/cargo-deps    #3340      +/-   ##
===================================================
- Coverage            61.83%   61.72%   -0.12%     
===================================================
  Files                  140      140              
  Lines                12318    12318              
  Branches              1617     1617              
===================================================
- Hits                  7617     7603      -14     
- Misses                3989     4002      +13     
- Partials               712      713       +1     

see 2 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 60f33e9...25d0c84. Read the comment docs.

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

Base automatically changed from levi/cargo-deps to master July 28, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

☠️ do-not-merge/WIP profiling Relates to the Continuous Profiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants