Skip to content

chore(internal): migrate AgentResponse class to native#18115

Open
brettlangdon wants to merge 5 commits into
mainfrom
brettlangdon/native.agent.response
Open

chore(internal): migrate AgentResponse class to native#18115
brettlangdon wants to merge 5 commits into
mainfrom
brettlangdon/native.agent.response

Conversation

@brettlangdon
Copy link
Copy Markdown
Member

Description

Separated from #17626

Testing

Risks

Additional Notes

@brettlangdon brettlangdon requested review from a team as code owners May 15, 2026 16:11
@brettlangdon brettlangdon added the changelog/no-changelog A changelog entry is not required for this PR. label May 15, 2026
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

cit-pr-commenter-54b7da Bot commented May 15, 2026

Codeowners resolved as

src/native/data_pipeline/agent_response.rs                              @DataDog/apm-core-python

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c473732a0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/native/data_pipeline/agent_response.rs Outdated
@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented May 15, 2026

Benchmarks

Benchmark execution time: 2026-05-20 16:14:11

Comparing candidate commit dcf5812 in PR branch brettlangdon/native.agent.response with baseline commit 71279ab in branch main.

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

scenario:iastaspects-index_aspect

  • 🟥 execution_time [+9.508µs; +12.615µs] or [+7.800%; +10.349%]

scenario:iastaspects-stringio_aspect

  • 🟥 execution_time [+527.650µs; +584.627µs] or [+13.764%; +15.251%]

scenario:iastaspectsospath-ospathbasename_aspect

  • 🟥 execution_time [+84.095µs; +99.474µs] or [+20.096%; +23.771%]

scenario:span-start

  • 🟥 execution_time [+1.390ms; +1.547ms] or [+8.903%; +9.907%]

scenario:tracer-small

  • 🟥 execution_time [+26.551µs; +28.466µs] or [+7.882%; +8.451%]

@datadog-official
Copy link
Copy Markdown
Contributor

datadog-official Bot commented May 20, 2026

Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: dcf5812 | Docs | Datadog PR Page | Give us feedback!

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