feat(trace-protobuf): Add two fields to ClientGroupedStats#1630
feat(trace-protobuf): Add two fields to ClientGroupedStats#1630
Conversation
📚 Documentation Check Results📦
|
🔒 Cargo Deny Results✅ No issues found! 📦
|
BenchmarksComparisonBenchmark execution time: 2026-02-25 23:18:11 Comparing candidate commit c2fe7cb in PR branch Found 6 performance improvements and 6 performance regressions! Performance is the same for 45 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
scenario:credit_card/is_card_number/ 378282246310005
scenario:credit_card/is_card_number/378282246310005
scenario:normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000...
scenario:normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters
scenario:profile_add_sample2_frames_x1000
scenario:profile_add_sample_frames_x1000
scenario:tags/replace_trace_tags
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. |
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
|
7c941c6 to
fb53fc8
Compare
Clippy Allow Annotation ReportComparing clippy allow annotations between branches:
Summary by Rule
Annotation Counts by File
Annotation Stats by Crate
About This ReportThis report tracks Clippy allow annotations for specific rules, showing how they've changed in this PR. Decreasing the number of these annotations generally improves code quality. |
…b.rs - Align stats.proto with datadog-agent (use @inject_tag comment instead of gogoproto annotation, remove gogo.proto import, update field comments) - Update DATADOG_AGENT_TAG to bdcdd8cf1ba4090a29b96d5669cfab5dd81814b1 - Regenerate pb.rs with service_source and span_derived_primary_tags fields - Add serde(default) attributes for new fields in build.rs - Fix pb_test.rs to include new fields in ClientGroupedStats initializer Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…_stats Add service_source and span_derived_primary_tags to the ClientGroupedStats initializer in aggregation.rs following the addition of these fields to the protobuf definition. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add intermediate *const () pointer cast before usize cast for signal handler function pointers, as required by the function_casts_as_integer clippy lint. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…dling Replace is_some() + unwrap() pattern with idiomatic if let Some(...) in config.rs and entry.rs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ior.rs Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.