LLM Obs SDK use context API for parent children span linkage by gary-huang · Pull Request #8711 · DataDog/dd-trace-java · GitHub
Skip to content

LLM Obs SDK use context API for parent children span linkage#8711

Merged
gary-huang merged 27 commits into
gary/llmobs-sdk-mergefrom
gary/use-ctx-api
Jul 8, 2025
Merged

LLM Obs SDK use context API for parent children span linkage#8711
gary-huang merged 27 commits into
gary/llmobs-sdk-mergefrom
gary/use-ctx-api

Conversation

@gary-huang

Copy link
Copy Markdown
Contributor

What Does This Do

Motivation

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@pr-commenter

pr-commenter Bot commented Apr 21, 2025

Copy link
Copy Markdown

@gary-huang gary-huang changed the title Gary/use ctx api LLM Obs SDK use context API for parent children span linkage Jun 26, 2025
@nayeem-kamal nayeem-kamal requested a review from dougqh July 2, 2025 17:49
@nayeem-kamal nayeem-kamal marked this pull request as ready for review July 3, 2025 20:09
@nayeem-kamal nayeem-kamal self-requested a review July 3, 2025 20:10
@gary-huang gary-huang requested review from a team as code owners July 8, 2025 16:22
@gary-huang gary-huang requested review from manuel-alvarez-alvarez, randomanderson and shatzi and removed request for a team July 8, 2025 16:22
@gary-huang gary-huang marked this pull request as draft July 8, 2025 16:23
@gary-huang gary-huang force-pushed the gary/use-ctx-api branch 2 times, most recently from 06e220a to 7c16dd2 Compare July 8, 2025 16:28
@gary-huang gary-huang marked this pull request as ready for review July 8, 2025 19:33
@gary-huang gary-huang merged this pull request into gary/llmobs-sdk-merge Jul 8, 2025
497 of 503 checks passed
@gary-huang gary-huang deleted the gary/use-ctx-api branch July 8, 2025 19:33
gary-huang added a commit that referenced this pull request Jul 9, 2025
* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* impl llmobs agent and llmobs apis

* use new ctx api to track parent span

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* ran spotless

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>
nayeem-kamal added a commit that referenced this pull request Jul 9, 2025
* add APIs for llm obs sdk (#8135)

* add APIs for llm obs

* add llm message class to support llm spans

* follow java convention of naming Id instead of ID

* add codeowners

* implement LLM Obs SDK spans APIs (#8390)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* spotless

* add APIs for llm obs sdk (#8135)

* add APIs for llm obs

* add llm message class to support llm spans

* follow java convention of naming Id instead of ID

* add codeowners

* rename ID to Id according to java naming conventions

* Undo change to integrations-core submodule

* fix build gradle

* rm empty line

* fix test

* LLM Obs SDK Mapper (#8372)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>

* updated to master submodule

* LLM Obs SDK use context API for parent children span linkage (#8711)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* impl llmobs agent and llmobs apis

* use new ctx api to track parent span

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* ran spotless

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>

* LLM Obs SDK evaluation metrics submission (#8688)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* impl llmobs agent and llmobs apis

* use new ctx api to track parent span

* add api for evals

* working impl supporting both agentless and agent

* handle null tags and default to default ml app if null or empty string provided in the override

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* ran spotless

* ran spotless

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>

* fix CODEOWNERS

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>
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.

2 participants