Skip to content

Conversation

taegyunkim
Copy link
Contributor

Checklist

  • PR author has checked that all the criteria below are met
  • The PR description includes an overview of the change
  • The PR description articulates the motivation for the change
  • The change includes tests OR the PR description describes a testing strategy
  • The PR description notes risks associated with the change, if any
  • Newly-added code is easy to change
  • The change follows the library release note guidelines
  • The change includes or references documentation updates if necessary
  • Backport labels are set (if applicable)

Reviewer Checklist

  • Reviewer has checked that all the criteria below are met
  • Title is accurate
  • All changes are related to the pull request's stated goal
  • Avoids breaking API changes
  • Testing strategy adequately addresses listed risks
  • Newly-added code is easy to change
  • Release note makes sense to a user of the library
  • If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
  • Backport labels are set in a manner that is consistent with the release branch maintenance policy

Copy link
Contributor

github-actions bot commented Sep 12, 2025

CODEOWNERS have been resolved as:

ddtrace/profiling/__init__.pyi                                          @DataDog/profiling-python
ddtrace/profiling/product.py                                            @DataDog/profiling-python
ddtrace/bootstrap/preload.py                                            @DataDog/apm-core-python
ddtrace/profiling/__init__.py                                           @DataDog/profiling-python
ddtrace/profiling/collector/_lock.py                                    @DataDog/profiling-python
ddtrace/profiling/collector/_task.pyx                                   @DataDog/profiling-python
ddtrace/profiling/collector/memalloc.py                                 @DataDog/profiling-python
ddtrace/profiling/collector/pytorch.py                                  @DataDog/profiling-python
ddtrace/profiling/collector/stack.pyx                                   @DataDog/profiling-python
ddtrace/profiling/collector/threading.py                                @DataDog/profiling-python
ddtrace/profiling/profiler.py                                           @DataDog/profiling-python
pyproject.toml                                                          @DataDog/python-guild
tests/profiling/gevent_fork.py                                          @DataDog/profiling-python
tests/profiling/simple_program_fork.py                                  @DataDog/profiling-python
tests/profiling/test_main.py                                            @DataDog/profiling-python
tests/profiling/test_uwsgi.py                                           @DataDog/profiling-python
tests/profiling/uwsgi-app.py                                            @DataDog/profiling-python
tests/profiling_v2/collector/test_threading.py                          @DataDog/profiling-python
tests/profiling_v2/simple_program_fork.py                               @DataDog/profiling-python
tests/profiling_v2/test_main.py                                         @DataDog/profiling-python
tests/profiling_v2/test_uwsgi.py                                        @DataDog/profiling-python
tests/telemetry/test_writer.py                                          @DataDog/apm-python

@taegyunkim taegyunkim added the changelog/no-changelog A changelog entry is not required for this PR. label Sep 12, 2025
Copy link
Contributor

github-actions bot commented Sep 12, 2025

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 277 ± 5 ms.

The average import time from base is: 274 ± 3 ms.

The import time difference between this PR and base is: 3.7 ± 0.2 ms.

Import time breakdown

The following import paths have grown:

ddtrace.auto 1.843 ms (0.66%)
ddtrace.bootstrap.sitecustomize 1.843 ms (0.66%)
ddtrace.bootstrap.preload 1.667 ms (0.60%)
ddtrace.internal.remoteconfig.worker 0.109 ms (0.04%)
ddtrace.internal.remoteconfig._connectors 0.025 ms (0.01%)
ddtrace._trace.trace_handlers 0.175 ms (0.06%)
ddtrace.contrib.internal.botocore.constants 0.080 ms (0.03%)
ddtrace.contrib.internal.botocore 0.080 ms (0.03%)

The following import paths have shrunk:

ddtrace.auto 1.437 ms (0.52%)
ddtrace.bootstrap.sitecustomize 0.767 ms (0.28%)
ddtrace.bootstrap.preload 0.683 ms (0.25%)
ddtrace.internal.remoteconfig.client 0.592 ms (0.21%)
ddtrace.debugging._import 0.091 ms (0.03%)
ddtrace.debugging._function.discovery 0.091 ms (0.03%)
ddtrace._trace.trace_handlers 0.084 ms (0.03%)
ddtrace.ext.azure_servicebus 0.084 ms (0.03%)
ddtrace 0.670 ms (0.24%)
ddtrace.internal._unpatched 0.027 ms (0.01%)
json 0.027 ms (0.01%)
json.decoder 0.027 ms (0.01%)
re 0.027 ms (0.01%)
enum 0.027 ms (0.01%)
types 0.027 ms (0.01%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants