Skip to content

OTel Metric templates are direct protobuf types#1350

Merged
blt merged 5 commits intomainfrom
blt/otel_metric_templates_are_direct_protobuf_types
May 16, 2025
Merged

OTel Metric templates are direct protobuf types#1350
blt merged 5 commits intomainfrom
blt/otel_metric_templates_are_direct_protobuf_types

Conversation

@blt
Copy link
Copy Markdown
Collaborator

@blt blt commented May 12, 2025

What does this PR do?

This commit removes the intermediary template types in favor of producing
directly into serializable types.

Motivation

REF SMPTNG-569

@blt blt added the no-changelog label May 12, 2025 — with Graphite App
@blt blt marked this pull request as ready for review May 12, 2025 19:52
@blt blt requested a review from a team as a code owner May 12, 2025 19:52
@datadog-datadog-prod-us1
Copy link
Copy Markdown

datadog-datadog-prod-us1 bot commented May 12, 2025

Datadog Summary

✅ Code Quality    ✅ Code Security    ✅ Dependencies


Was this helpful? Give us feedback!

@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 935e384 to 49549ec Compare May 12, 2025 21:43
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch 2 times, most recently from 9a171ae to ddc5dfa Compare May 12, 2025 21:46
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch 2 times, most recently from 05fca81 to f7db081 Compare May 12, 2025 22:49
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch 2 times, most recently from 89e2f4a to e87c0f3 Compare May 12, 2025 23:05
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from f7db081 to 1134de9 Compare May 12, 2025 23:05
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from e87c0f3 to 62810d6 Compare May 12, 2025 23:08
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch 2 times, most recently from 3f73728 to c1554c2 Compare May 12, 2025 23:09
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 62810d6 to 0c38025 Compare May 12, 2025 23:10
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from c1554c2 to b705974 Compare May 12, 2025 23:45
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 0c38025 to 2ba374d Compare May 12, 2025 23:45
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from b705974 to c6a82e1 Compare May 13, 2025 00:27
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 2ba374d to 3b2bf3e Compare May 13, 2025 00:28
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from c6a82e1 to f5e7ff3 Compare May 13, 2025 18:29
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 3b2bf3e to 619f189 Compare May 13, 2025 18:29
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 619f189 to 87fa48b Compare May 14, 2025 16:05
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from f5e7ff3 to 60f66df Compare May 14, 2025 16:05
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 87fa48b to f24978e Compare May 14, 2025 16:15
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 60f66df to 9b3a931 Compare May 14, 2025 16:15
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 30286ee to 8f095f4 Compare May 14, 2025 22:36
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 0570d75 to 986d6dc Compare May 15, 2025 15:48
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 8f095f4 to 9a1e8e2 Compare May 15, 2025 15:48
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 986d6dc to 810c9be Compare May 15, 2025 15:59
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 9a1e8e2 to 585b425 Compare May 15, 2025 16:00
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 810c9be to 533936d Compare May 15, 2025 17:25
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 585b425 to 8682b76 Compare May 15, 2025 17:26
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from 533936d to cc26354 Compare May 15, 2025 22:59
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 8682b76 to 0486539 Compare May 15, 2025 22:59
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch from cc26354 to dbb3152 Compare May 15, 2025 23:15
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 0486539 to 1d248ee Compare May 15, 2025 23:15
@blt blt force-pushed the blt/optimization_passes_for_otel_metrics_payload_generation branch 2 times, most recently from 40ad9a2 to 68c97ab Compare May 15, 2025 23:29
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 1d248ee to 88cb8ba Compare May 15, 2025 23:29
@blt blt changed the base branch from blt/optimization_passes_for_otel_metrics_payload_generation to graphite-base/1350 May 15, 2025 23:43
@preinlein preinlein changed the base branch from graphite-base/1350 to main May 15, 2025 23:44
blt added 4 commits May 15, 2025 23:44
This commit removes the intermediary template types in favor of producing
directly into serializable types.

REF SMPTNG-569

Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
I neglected to pop the too-big resource_metrics off the request once we
detected that we were past the limit. Fixed.

Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from 88cb8ba to e3a8386 Compare May 15, 2025 23:44
Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
@blt blt force-pushed the blt/otel_metric_templates_are_direct_protobuf_types branch from e3a8386 to 867ab2b Compare May 15, 2025 23:44
@blt blt merged commit 916579d into main May 16, 2025
21 checks passed
Copy link
Copy Markdown
Collaborator Author

blt commented May 16, 2025

Merge activity

  • May 15, 8:01 PM EDT: @blt merged this pull request with Graphite.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants