DataDog / dd-trace-py

Datadog Python APM Client
https://ddtrace.readthedocs.io/
Other
553 stars 416 forks source link

chore(profiling): build dd_wrapper once and share that #11472

Open taegyunkim opened 5 days ago

taegyunkim commented 5 days ago

Checklist

Reviewer Checklist

github-actions[bot] commented 5 days ago

CODEOWNERS have been resolved as:

ddtrace/internal/datadog/profiling/cmake/AnalysisFunc.cmake             @DataDog/profiling-python
ddtrace/internal/datadog/profiling/crashtracker/CMakeLists.txt          @DataDog/profiling-python
ddtrace/internal/datadog/profiling/dd_wrapper/CMakeLists.txt            @DataDog/profiling-python
ddtrace/internal/datadog/profiling/ddup/CMakeLists.txt                  @DataDog/profiling-python
ddtrace/internal/datadog/profiling/stack_v2/CMakeLists.txt              @DataDog/profiling-python
setup.py                                                                @DataDog/python-guild
pr-commenter[bot] commented 5 days ago

Benchmarks

Benchmark execution time: 2024-11-20 21:07:29

Comparing candidate commit 95ca17c6ad117a18f7623bcbca187a302cbd55c8 in PR branch taegyunkim/ddwrapper-build with baseline commit c26fd4c9dffa1c8f083db729462d2b2286c60ddb in branch main.

Found 7 performance improvements and 0 performance regressions! Performance is the same for 381 metrics, 2 unstable metrics.

scenario:flasksimple-appsec-get

scenario:flasksimple-appsec-post

scenario:flasksimple-appsec-telemetry

scenario:flasksimple-tracer

scenario:flasksqli-appsec-enabled

scenario:flasksqli-iast-enabled

scenario:flasksqli-tracer-enabled