perf(profiling): reduce profiler arena memory footprint#18469
perf(profiling): reduce profiler arena memory footprint#18469taegyunkim wants to merge 6 commits into
Conversation
Codeowners resolved as |
|
BenchmarksBenchmark execution time: 2026-06-10 19:04:57 Comparing candidate commit 5a82f85 in PR branch Found 0 performance improvements and 6 performance regressions! Performance is the same for 615 metrics, 10 unstable metrics. scenario:iast_aspects-re_match_noaspect
scenario:iastaspects-index_aspect
scenario:iastaspects-ljust_aspect
scenario:iastaspects-stringio_noaspect
scenario:iastaspectsospath-ospathbasename_aspect
scenario:span-start
|
…-reduce-profiler-arena # Conflicts: # src/native/Cargo.toml
Description
Pins the
src/nativelibdatadog dependencies to commitb8f8c6de3c0028133d78996926d4777aa20f45c9from DataDog/libdatadog#2048 to pick up the profiler arena memory-footprint reduction.This also adapts the tracer flare native wrapper to the libdatadog remote-config parser registry API, which replaces
RemoteConfigDatawith parsed product data returned bydefault_registry().parse(...). The updated pin also includes the upstream crate rename fromdatadog-remote-configtolibdd-remote-config; dd-trace-py keeps the existing Rust import name via Cargo'spackage = "libdd-remote-config"alias.Testing
cd src/native && cargo check --features profilingscripts/run-tests --venv 803a341 tests/internal/test_tracer_flare_native.py -- -- tests/internal/test_tracer_flare_native.py -qscripts/run-tests --venv 803a341 tests/internal/test_tracer_flare.py -- -- tests/internal/test_tracer_flare.py::TracerFlareSubscriberTests::test_configuration_order_payload_is_skipped tests/internal/test_tracer_flare.py::TracerFlareSubscriberTests::test_process_flare_request_success -qscripts/lint fmt setup.pyscripts/lint spelling releasenotes/notes/profiling-libdatadog-reduce-profiler-arena-d781a459638124bd.yamlgit diff --checkcargo check --manifest-path src/native/Cargo.toml --lockedcargo check --manifest-path src/native/Cargo.toml --locked --all-featuresRisks
Additional Notes
Native review evidence:
cargo check --features profilingcompiled the native crate against the original libdatadog PR pin;cargo check --manifest-path src/native/Cargo.toml --locked --all-featurescompiled the native crate against the updatedb8f8c6de3c0028133d78996926d4777aa20f45c9pin.b8f8c6de3c0028133d78996926d4777aa20f45c9is associated with perf(profiling): reduce profiler arena memory footprint libdatadog#2048, confirmed itsrust-toolchain.tomlstill requires Rust 1.87.0, and adapted to the pinned commit's remote-config crate rename/API.