Skip to content

chore: Upgrade libdatadog and construct http client for traces#917

Merged
lym953 merged 5 commits intomainfrom
yiming.luo/upgrade-libdd
Nov 25, 2025
Merged

chore: Upgrade libdatadog and construct http client for traces#917
lym953 merged 5 commits intomainfrom
yiming.luo/upgrade-libdd

Conversation

@lym953
Copy link
Copy Markdown
Contributor

@lym953 lym953 commented Nov 7, 2025

Upgrade libdatadog. Including:

  • Rename a few creates:
    • ddcommon -> libdd-common
    • datadog-trace-protobuf -> libdd-trace-protobuf
    • datadog-trace-utils -> libdd-trace-utils
    • datadog-trace-normalization -> libdd-trace-normalization
    • datadog-trace-stats -> libdd-trace-stats
  • Use the new API to send traces, which takes in an http_client object instead of proxy url string

GitHub issue: #860
Jira: https://datadoghq.atlassian.net/browse/SLES-2499
Slack discussion: https://dd.slack.com/archives/C01TCF143GB/p1762526199549409

Comment thread bottlecap/src/traces/trace_flusher.rs
@lym953 lym953 force-pushed the yiming.luo/upgrade-libdd branch from afba974 to 84b22f2 Compare November 13, 2025 18:31
@lym953 lym953 marked this pull request as ready for review November 13, 2025 18:40
@lym953 lym953 requested a review from a team as a code owner November 13, 2025 18:40
@lym953 lym953 merged commit 8a81b38 into main Nov 25, 2025
39 checks passed
@lym953 lym953 deleted the yiming.luo/upgrade-libdd branch November 25, 2025 18:36
duncanpharvey pushed a commit that referenced this pull request Mar 10, 2026
Upgrade libdatadog. Including:
- Rename a few creates:
  - `ddcommon` -> `libdd-common`
  - `datadog-trace-protobuf` -> `libdd-trace-protobuf`
  - `datadog-trace-utils` -> `libdd-trace-utils`
  - `datadog-trace-normalization` -> `libdd-trace-normalization`
  - `datadog-trace-stats` -> `libdd-trace-stats`
- Use the new API to send traces, which takes in an http_client object
instead of proxy url string

GitHub issue:
#860
Jira: https://datadoghq.atlassian.net/browse/SLES-2499
Slack discussion:
https://dd.slack.com/archives/C01TCF143GB/p1762526199549409
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