Skip to content

Support sending OpenTelemetry logs over OTLP (http/protobuf)#11253

Open
mcculls wants to merge 9 commits intomasterfrom
mcculls/otlp-logs-proto
Open

Support sending OpenTelemetry logs over OTLP (http/protobuf)#11253
mcculls wants to merge 9 commits intomasterfrom
mcculls/otlp-logs-proto

Conversation

@mcculls
Copy link
Copy Markdown
Contributor

@mcculls mcculls commented May 1, 2026

Additional Notes

Builds on top of #11250

Contributor Checklist

Jira ticket: [PROJ-IDENT]

Note: Once your PR is ready to merge, add it to the merge queue by commenting /merge. /merge -c cancels the queue request. /merge -f --reason "reason" skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.

@mcculls mcculls requested review from a team as code owners May 1, 2026 08:57
@mcculls mcculls requested review from dougqh and removed request for a team May 1, 2026 08:57
@mcculls mcculls added the inst: opentelemetry OpenTelemetry instrumentation label May 1, 2026
@mcculls mcculls requested review from mhlidd and mtoffl01 and removed request for a team May 1, 2026 08:57
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: 48a6076f65

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread dd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated
Comment thread internal-api/src/main/java/datadog/trace/api/Config.java Outdated
Comment thread dd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsCollector.java Outdated
Comment thread dd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated
Comment thread dd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated
Copy link
Copy Markdown
Contributor

@dougqh dougqh left a comment

Choose a reason for hiding this comment

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

Overall, it looks good to me.

I've added a few performance suggestions both my own and from Claude.
I don't feel strongly that any of them must be addressed before this is merged.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants