Skip to content

Vlad/315 profiling dev tooling - #19260

Merged
vlad-scherbich merged 3 commits into
vlad/315-official-supportfrom
vlad/315-profiling-dev-tooling
Jul 24, 2026
Merged

Vlad/315 profiling dev tooling#19260
vlad-scherbich merged 3 commits into
vlad/315-official-supportfrom
vlad/315-profiling-dev-tooling

Conversation

@vlad-scherbich

@vlad-scherbich vlad-scherbich commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

prev: #19256 | next: #19258

Summary

Profiling bring-up scripts, compatibility baselines, Echion migration runbook (GAP-03).

Test plan

  • scripts/run-profiling-tests --check-only passes on 3.15 (when available)
  • Docs build / link check

@datadog-prod-us1-4

This comment has been minimized.

@vlad-scherbich
vlad-scherbich force-pushed the vlad/315-profiling-dev-tooling branch 2 times, most recently from cce4946 to 43e0460 Compare July 23, 2026 14:15
@pr-commenter

pr-commenter Bot commented Jul 23, 2026

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-07-23 14:17:42

Comparing candidate commit 672f90d in PR branch vlad/315-profiling-dev-tooling with baseline commit 528a085 in branch main.

Found 0 performance improvements and 4 performance regressions! Performance is the same for 615 metrics, 10 unstable metrics.

scenario:iastaspects-lstrip_aspect

  • 🟥 execution_time [+49.995µs; +54.331µs] or [+15.303%; +16.630%]

scenario:iastaspectsospath-ospathbasename_aspect

  • 🟥 execution_time [+59.440µs; +70.673µs] or [+13.478%; +16.025%]

scenario:iastaspectssplit-rsplit_aspect

  • 🟥 execution_time [+14.167µs; +17.160µs] or [+9.162%; +11.097%]

scenario:telemetryaddmetric-1-count-metric-1-times

  • 🟥 execution_time [+230.663ns; +263.989ns] or [+10.914%; +12.491%]

@vlad-scherbich
vlad-scherbich changed the base branch from main to vlad/ddtracepy-315-profiling-asyncio-monitoring July 23, 2026 14:18
@vlad-scherbich
vlad-scherbich force-pushed the vlad/ddtracepy-315-profiling-asyncio-monitoring branch from 4d06889 to 6e7f707 Compare July 23, 2026 14:31
@vlad-scherbich
vlad-scherbich force-pushed the vlad/315-profiling-dev-tooling branch from 43e0460 to 188e9f2 Compare July 23, 2026 14:32
Profiling bring-up scripts, compatibility baselines, Echion migration runbook,
and py315-stack PR navigation helpers (PR 12/14).
@vlad-scherbich
vlad-scherbich force-pushed the vlad/ddtracepy-315-profiling-asyncio-monitoring branch from 6e7f707 to 4b85b1d Compare July 24, 2026 01:25
@vlad-scherbich
vlad-scherbich force-pushed the vlad/315-profiling-dev-tooling branch from 188e9f2 to 233cb59 Compare July 24, 2026 01:25
Base automatically changed from vlad/ddtracepy-315-profiling-asyncio-monitoring to vlad/315-official-support July 24, 2026 01:25
@vlad-scherbich
vlad-scherbich merged commit 233cb59 into vlad/315-official-support Jul 24, 2026
4 of 6 checks passed
@vlad-scherbich
vlad-scherbich deleted the vlad/315-profiling-dev-tooling branch July 24, 2026 01:25
@cit-pr-commenter-54b7da

Copy link
Copy Markdown

Codeowners resolved as

.claude/skills/type-annotate/SKILL.md                                   @DataDog/apm-core-python
AGENTS.md                                                               @DataDog/apm-core-python

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7da Bot commented Jul 24, 2026

Copy link
Copy Markdown

Circular import analysis

⚠️ Existing circular imports

There are 33 circular imports that already exist on the base branch and have not been changed by this PR.

Show existing cycles (showing 5 of 33 shortest)
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.kafka -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.botocore -> ddtrace.internal.datastreams
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.kombu -> ddtrace.internal.datastreams
ddtrace.internal.core -> ddtrace._trace.span -> ddtrace.internal.core
ddtrace.internal.datastreams -> ddtrace.internal.datastreams.aiokafka -> ddtrace.internal.datastreams

To see all cycles, download the cycles-base.json and cycles-pr.json artifacts from this CI job and run:

uv run --script scripts/import-analysis/cycles.py compare cycles-base.json cycles-pr.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant