Conversation
BenchmarksComparisonBenchmark execution time: 2026-02-06 20:07:52 Comparing candidate commit e132864 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1450 +/- ##
==========================================
- Coverage 71.20% 71.18% -0.03%
==========================================
Files 424 424
Lines 69802 69864 +62
==========================================
+ Hits 49700 49730 +30
- Misses 20102 20134 +32
🚀 New features to boost your workflow:
|
79ff759 to
bd8abfc
Compare
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
60395d1 to
4650827
Compare
3c1a633 to
2e7531b
Compare
ivoanjo
left a comment
There was a problem hiding this comment.
Looks very reasonable! Left a few small comments + this would probably benefit from a pass from a real rustacean as I won't catch the more minute details there ;)
| /// When profiles are reset, the period needs to be preserved. This | ||
| /// stores it in a way that does not depend on the string table. | ||
| owned_period: Option<owned_types::Period>, |
There was a problem hiding this comment.
I'm curious, why change the period as well?
There was a problem hiding this comment.
Now that we have a structured enum, might as well use it
2e7531b to
16bc2b2
Compare
📚 Documentation Check Results📦
|
🔒 Cargo Deny Results📦
|
16bc2b2 to
d401aa8
Compare
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
devflow unqueued this merge request: It did not become mergeable within the expected time |
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
The expected merge time in
PR can't be merged according to github policy. It's probably because:
|
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
devflow unqueued this merge request: It did not become mergeable within the expected time |
# What does this PR do? Reverts #1478 # Motivation As per the event-platform-intake team: // Note: We don't set Content-Type for file attachments in the multipart form. // The intake backend treats all attachments as raw bytes (application/octet-stream) // and detects compression by reading magic bytes (gzip/zstd/etc headers). // Content-Type is only meaningful for the main "event" part (set to application/json). // Attachments are not forwarded beyond intake, so their MIME types are not needed. # Additional Notes Anything else we should know when reviewing? # How to test the change? Describe here in detail how the change can be validated. Co-authored-by: daniel.schwartznarbonne <daniel.schwartznarbonne@datadoghq.com>
…cripts (#1506) # What does this PR do? Attempt to fix a potential race condition with the [all-checks](https://github.com/DataDog/libdatadog/blob/b44bb77dc7e7dcfd8e47d9e8c2bbe1d3cfa894f6/.github/workflows/all-checks.yml) workflow and [report_gitlab_CI_status](https://github.com/DataDog/libdatadog/blob/b44bb77dc7e7dcfd8e47d9e8c2bbe1d3cfa894f6/.gitlab-ci.yml#L42) `report_gitlab_CI_status` is `post` stage and doesn't get created until after dependent jobs run. If all gitlab benchmark and github actions successfully complete, but `report_gitlab_CI_status` hasn't been created yet, then all-checks can pass because all-checks doesn't see anything pending. The `dd-gitlab/default-pipeline` check has been enabled for this repo by ci-infra. This should solve our problems and remove the need for custom scripts. It will pass when the Gitlab pipeline completes. This check will be added to the branch protection rule for main. all-checks will just be used for github steps. # Motivation all-checks passed on #1428 , despite tests failing on the alpine builds in gitlab. # Additional Notes Anything else we should know when reviewing? # How to test the change? Describe here in detail how the change can be validated. Co-authored-by: edmund.kump <edmund.kump@datadoghq.com>
What does this PR do?
Takes sample types by enum rather than as tuples of strings.
Motivation
Additional Notes
Added experimental enum variants to allow users to experiment without needing to update libdatadog.
How to test the change?
Describe here in detail how the change can be validated.